Repository navigation
docs(agents): principles-only os-dev definition — lessons distilled in place, no issue-ID citations - #7938
Merged
Conversation
…n place, no issue-ID citations The dev-agent definition is rewritten to the same standard as the pm-dispatch principles rewrite (maintainer rulings 2026-08-12: 「只需要说原则,不需要写 细节」;「保留 issue id没有意义」): every incident-backed rule becomes a self-contained lesson (failure mode + discipline + boundary), hook-enforced details keep one principle line each, operational lookups (toolchain traps, report contract, gate families) stay as data. 686 → 356 lines; zero issue-ID citations (the pm-skill ID lint's legacy waiver for this file self-expires at zero). The three-axis decision frame keeps its frame-sync anchors verbatim. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01W3v2G9dvcfxkC4NsZ4JCE9
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
github-actions
Bot
added
size/l
documentation
os-zhuang
added
the
skip-changeset
Claude
…the frame-sync fixture The self-test's extraction-failure fixture removes the declaring sentence by literal single-line match; the rewrap had split it across a line break. The gate's own anchor matching is whitespace-tolerant — only the fixture needs the head contiguous. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01W3v2G9dvcfxkC4NsZ4JCE9
os-zhuang
marked this pull request as ready for review
August 12, 2026 07:42
Contributor
Author
|
同意合并 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #7903
ADR-class by the recorded governance note (an agent definition governs every dev session's behavior): draft, human merge only — ⛔ no queue entry, no auto-merge. Independent of the #7885 PR (PR₁): branched off current
origin/main, no file overlap, either merge order works (PR₁'s ID-lint carries a self-expiring exact-count waiver for this file).What this is
.claude/agents/os-dev.mdrewritten to the same writing standard as the pm-dispatch principles rewrite (maintainer rulings 2026-08-12: 「只需要说原则,不需要写细节」;「处理 issue 时犯的错应该总结成经验,保留 issue id没有意义」): 686 → 356 lines, zero issue-ID citations (81 removed; every lesson now self-contained — failure mode + discipline + boundary), maintainer rulings kept as date + verbatim quote.The three-way sorting rule applied:
scripts/pm/os-regen-merge.sh), nul-byte gate (its header cited as the authority instead of re-arguing the harms).flocklock path, heap cap, worker caps), the attribution-footer forms.Everything binding survived as a rule: the six ground rules, resource discipline (foreground pipeline, PID-only kills, unforced worktree removal), local verification scope, the standard clauses (build-first, prefix filter direction, reverse verification with its three directions, spec anchor / MERGE-state trap, rejection-envelope
code+status, key-vs-value criterion, fixture triage's three dispositions + consumption-radius sweep), Definition of done (Fixes/Part-of rule, skip-changeset read-back, report-at-draft-PR-time with the platform-subscription override), terminating cleanly (report twice GitHub-first with marker read-back, monitors never outlive their subject, silence-is-not-success + the PM probe backstop), the three-axis escalation frame (frame-sync anchors verbatim), and byte/sanitizer discipline.Deviation to review
356 lines vs the ~200–300 target. The remaining ~60 lines over target are load-bearing rules I judged non-droppable under the no-silent-semantic-loss red line (mostly the standard clauses and definition-of-done, which dispatch prompts deliberately do NOT repeat — this file is their only home). If you want it tighter, the candidates are named sections, not sentence trims — say which section may lose semantics.
Gate status (honest, at draft-PR time)
Local, all green:
check:agent-model-declared(pin kept),check:skill-frame-sync(4 copies isomorphic; anchors preserved verbatim),check:nul-bytes,check:doc-authoring; zero#[0-9]{3,}matches (PR₁'s new lint goes fully strict on this file once both land). CI:in_progressat report time — the PM owns convergence.No changeset:
.claude/-only change (skip-changesetapplied).Generated by Claude Code