Skip to content

Docs: AUTHORING.md — manifest parser strips unknown fields, doesn't reject them - #869

Merged
selfcontained merged 1 commit into
mainfrom
agt_526a4dc342f1/job-doc-ter-2e03a89b
Aug 3, 2026
Merged

Docs: AUTHORING.md — manifest parser strips unknown fields, doesn't reject them#869
selfcontained merged 1 commit into
mainfrom
agt_526a4dc342f1/job-doc-ter-2e03a89b

Conversation

@selfcontained

Copy link
Copy Markdown
Owner

Nightly docs audit (2026-08-02, scheduled job).

Audited this run

  • Diff since 5525021 (Docs: fix operations-runbook drift — artifact-only updates, real release-workflow shape, cursor in preflight #865Cover use-release-stream: applyStreamEvent reducer and stream wiring #868): refactor/test-only, no doc-bearing changes.
  • docs-pane Notifications section vs notification-settings.tsx + split section components, notification-runtime.ts, focus-tracker.ts, use-agent-focus.ts, slack.ts, agent-lifecycle-tools.ts: all claims verified accurate (3s web-notify ack fallback, 15s heartbeat / 30s TTL focus suppression, 4 status cues + mobile tap, Tips & Guidance controls, dispatch_notify params/limits, persona agents lacking the tool).
  • docs-pane Repo Tools section vs repo-tools.ts, mcp/server.ts tool sets + toolsRoot, routes/mcp.ts scope, lifecycle-hooks.ts, setup-script.ts: all claims verified accurate (repo_ prefix, dot sanitization, param→flag mapping, effective agent/job scopes, stop hook 15s best-effort, DISPATCH_AGENT_ID, ~/.dispatch/env).

Changed

  • release-notes/AUTHORING.md: the migration-manifest section claimed fields outside the V1 schema "will fail validation". UpdateMigrationManifestSchema is a plain z.object — unknown keys are silently stripped, never rejected — and nothing else (bin scripts, CI) validates manifests strictly. Reworded so authors don't rely on an error that never fires.

Deferred / noted

  • Stale backlog item removed from Brain: "AUTHORING.md doesn't cover migration manifests" — it does now (Migration manifests section exists and its other claims verified against update-migrations.ts / update-migrations-evaluator.ts / assisted-update.ts).
  • Next run: docs-pane Worktrees section vs apps/server/src/shared/git/worktree.ts after the Consolidate duplicated git helpers into shared git-context #867 git-context consolidation.

🤖 Generated with Claude Code

The migration-manifest section of AUTHORING.md claimed extra fields
"will fail validation", but UpdateMigrationManifestSchema is a plain
z.object — unknown keys are silently stripped, never rejected. Reword
so authors don't rely on a validation error that never fires.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@selfcontained
selfcontained merged commit de5461e into main Aug 3, 2026
1 check passed
@selfcontained
selfcontained deleted the agt_526a4dc342f1/job-doc-ter-2e03a89b branch August 3, 2026 04:10
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