Skip to content

docs: simplify agent context and resolve conflicting guidance - #2287

Merged
thymikee merged 2 commits into
mainfrom
docs/agent-context-follow-through
Sep 5, 2026
Merged

docs: simplify agent context and resolve conflicting guidance#2287
thymikee merged 2 commits into
mainfrom
docs/agent-context-follow-through

Conversation

@thymikee

@thymikee thymikee commented Sep 5, 2026

Copy link
Copy Markdown
Member

Summary

Remove generic execution advice, basic CLI tutorials, duplicated checklists, and rhetorical explanations from contributor context. Apply the GPT-6 Astra guidance by narrowing instructions to repository-specific constraints and resolving conflicting rules.

  • Align immediate skill startup with conditional help lookup, parallel work with distinct devices, and CI requirements with selected checks.
  • Clarify publication authorization, dependent PR publication versus merge-readiness, and validation after rebasing.
  • Correct misleading gate/sandbox failure diagnoses, nonexistent depgraph-to-gate input, and pure-test-move verification. Preserve non-obvious constraint comments.

Seven context files changed. Scope covers contributor guidance; the reviewed glossary and packaged skills remain unchanged. Compared with main, AGENTS.md shrinks from 9,994 to 7,805 bytes, and root context plus routed procedures shrink 11.7%. Existing context budgets and gate definitions are unchanged.

Validation

Validated commit: 501426464c587f4efb6ff6028ba4ffc34b99821c.

Repository formatting, diff checks, referenced local-path checks, and the required affected gate passed (all four guidance contracts). The gate ran with host process-inspection access required by its harness. Manually reviewed instruction scope and cross-file consistency against the glossary, four packaged skills, and all nine routed procedures; checked affected-selector behavior against code. Documentation only, so runtime tests do not apply. CI on the updated PR head remains authoritative.

@thymikee

thymikee commented Sep 5, 2026

Copy link
Copy Markdown
Member Author

The new AGENTS.md precedence rule is unsafe: saying user instructions “take precedence over repository and skill guidance” invites agents to bypass binding repository constraints such as required gates, device/session ownership, worktree safety, and authorization boundaries.

Please narrow it: user requests define task intent and override discretionary guidance only; system/developer requirements and repository hard rules remain binding, except where a rule explicitly makes user authorization the condition for proceeding. On conflict, identify the exact constraint rather than silently bypassing it.

@thymikee thymikee changed the title docs: clarify agent task scope and validation docs: simplify agent context and resolve conflicting guidance Sep 5, 2026
@thymikee
thymikee merged commit 0da105e into main Sep 5, 2026
8 checks passed
@thymikee
thymikee deleted the docs/agent-context-follow-through branch September 5, 2026 18:04
@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-09-05 18:05 UTC

thymikee added a commit that referenced this pull request Sep 5, 2026
* origin/main:
  perf: bundle runtime dependencies and report full install size (#2310)
  ci: avoid unrelated Apple runner cache invalidation (#2303)
  fix(web): preserve the backend ref so snapshot refs match actionable refs (#2283)
  test(daemon): session-open-url-prewarm through the request seam (#2304)
  test(daemon): session-devices-batch-runtime through the request seam (#2305)
  chore(gates): layering baselines ratchet against merge-base (#2299)
  test(daemon): one typed conformance helper for the daemon runtime suites (#2298)
  chore(layering): derive the contracts export inventory from package.json (#2297)
  perf: bundle tar-stream to reduce install footprint (#2286)
  docs: simplify agent context and resolve conflicting guidance (#2287)
  refactor(cli): let help resolve command aliases itself and retire R12 (#2293)
thymikee added a commit that referenced this pull request Sep 5, 2026
* origin/main:
  perf: bundle runtime dependencies and report full install size (#2310)
  ci: avoid unrelated Apple runner cache invalidation (#2303)
  fix(web): preserve the backend ref so snapshot refs match actionable refs (#2283)
  test(daemon): session-open-url-prewarm through the request seam (#2304)
  test(daemon): session-devices-batch-runtime through the request seam (#2305)
  chore(gates): layering baselines ratchet against merge-base (#2299)
  test(daemon): one typed conformance helper for the daemon runtime suites (#2298)
  chore(layering): derive the contracts export inventory from package.json (#2297)
  perf: bundle tar-stream to reduce install footprint (#2286)
  docs: simplify agent context and resolve conflicting guidance (#2287)
  refactor(cli): let help resolve command aliases itself and retire R12 (#2293)
  refactor(commands): retire the navigation-only type projection (#2294)
  feat(runtime): route managed leases through contained transports (#2285)
  refactor(contracts): build unavailable runtime facts once (#2291)
  refactor(cli): derive the common flag readers from the common-field table (#2292)
  feat(daemon): add managed allocation operation journal (#2284)
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.

1 participant