Skip to content

Add project rules to AGENTS.md and the verify and add-surface skills - #2

Open
masaok wants to merge 1 commit into
engineering-practicesfrom
agent-instructions
Open

masaok wants to merge 1 commit into
engineering-practicesfrom
agent-instructions

Conversation

@masaok

@masaok masaok commented Oct 1, 2026

Copy link
Copy Markdown
Owner

Why

Agents need the project rules where they are loaded on every session, and the two repeatable procedures as skills. The rules say instruction changes get a pull request of their own, so these files are split from #1.

Scope

  • AGENTS.md gains the project rules, the directory table and the command table.
  • .claude/skills/verify and .claude/skills/add-surface.

Stacked on #1. Merge #1 first.

Tradeoffs

No eval run is attached. AGENTS.md asks for a run of the scenarios in docs/agent-evals.md with any instruction change. None has been logged yet. This pull request introduces the instructions, so there is no earlier behavior to compare against.

Verification

VERIFIED. pnpm verify on Node 24.21.0 on this branch: 12 tests pass, 2 surfaces requested, 0 failures.

NOT VERIFIED: the agent eval scenarios.

https://claude.ai/code/session_01WTtRZcdke1yYdYUAanaf7V

Copilot AI balanced review requested due to automatic review settings October 1, 2026 23:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@vercel

vercel Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
costexplorer Ready Ready Preview Oct 1, 2026 11:57pm UTC

Request Review

This branch was successfully deployed

1 active deployment
Preview — 677af2e7 Deployed Oct 1, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants