Skip to content

docs: full sync sweep v0.17.1→v0.18.1 + rolling anchor - #416

Merged
initializ-mk merged 7 commits into
mainfrom
docs/full-sync-sweep
Aug 21, 2026
Merged

docs: full sync sweep v0.17.1→v0.18.1 + rolling anchor#416
initializ-mk merged 7 commits into
mainfrom
docs/full-sync-sweep

Conversation

@initializ-mk

Copy link
Copy Markdown
Contributor

Full documentation sweep — everything merged v0.17.1 → v0.18.1 (120 commits / ~42 PRs), not just one branch. Driven by 5 parallel domain audits (MCP+auth, security/egress/governance, CLI/skills, LLM/runtime/memory, channels/audit/observability) that mapped each merged feature to its docs and flagged MISSING / STALE.

Also introduces a rolling anchor (docs/sync-docs-state.md) so future /sync-docs runs are incremental — the next sweep only diffs 2f0649e..main instead of re-scanning history.

Gaps found and filled (17 files)

Reference / schema (the biggest gaps — the canonical reference lagged the feature waves):

Audit catalog:

Stale corrections:

New / expanded sections:

Verification

Broken-link check clean across all changed docs. .claude/skills/forge.md TOC unaffected (row/prose edits only). Every claim traced to code by the audits (file:line in the audit trail).

What's intentionally NOT here

Pure bug fixes with no user-facing doc surface (e.g. Windows OAuth launch #312) and features already documented + verified accurate (forge try, skill import, data-part prompt, session-recovery dedup, SOCKS5 egress, DEFER approvers, denied_command_patterns).

@initializ-mk initializ-mk left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Review — docs sync sweep (v0.17.1 → v0.18.1)

Docs-only (all 17 files .md); CI green including the broken-link check. For a docs PR the risk is a wrong claim, so I spot-checked the highest-value ones — security-relevant, schema, and agent-facing — against the code (leaning on my reviews of these features and verifying two I hadn't). Everything I traced is accurate:

  • pdp: schemaenabled/endpoint/fail: closed, "env-expanded at load" (the exact #399 fix), single-decision-source, and precisely "every governed tool call" (correctly reflects the #401 builtins-skip, not a lazy "every tool call"). ✓
  • apis: schemabase_url (allowlisted), token_env "oauth rejected for apis", operations[].{method,path} with {account_id} templating (#400). ✓
  • Egress allowed_tcp / allowed_private_cidrs; defer detach #402 correctly replaces the stale "abandoned / cap ≤6m"; delegated-token 5m TTL. ✓
  • MCP tool-name regex ^[a-zA-Z0-9_-]{1,128}$ (#370) — present in forge-core/validate/mcp_config.go. ✓
  • Rolling anchor 2f0649e resolves and is conservatively set to the audited HEAD, so the next incremental sweep diffs correctly. ✓

Scope note (transparency, not a finding): this is a spot-check weighted to the security/schema/agent-facing claims — 100% accurate in my sample — not a line-by-line trace of all 258 lines; the prose-heavy channels/memory/observability details I sampled but didn't exhaustively verify. Confidence is high given the sample and the audits' file:line trail.

One high-value correction called out inline. Approve-grade — nothing blocking.

Comment thread .claude/skills/forge.md
### 12.11 Governance framework R1–R10 (#216 umbrella)

Six MUST + three SHOULD requirements from an agent-runtime governance framework, complete on `main` after #245 / #246 / #247 / #248 land (R4c is the last piece). **R10 (delegated identity) is a proposed fourth SHOULD** — not yet implemented; see #317 / #318.
Six MUST + three SHOULD requirements from an agent-runtime governance framework, complete on `main` after #245 / #246 / #247 / #248 land (R4c is the last piece). **R10 (delegated identity) is now IMPLEMENTED** — delegated per-user MCP auth (`auth.type: user`) with a per-user connection pool, lazy consent, and an auth-required gate shipped via #317 / #327 / #329 / #330 / #331 / #332 / #344, plus Slack consent delivery (#343 / #345) and agent-principal 2LO (#324 / #325). See `docs/mcp/configuration.md` and `docs/mcp/delegated-consent.md`.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Verified this agent-facing correction is accurate — and it's the most valuable edit in the sweep, because the old text was actively misleading the agent's own reasoning:

Good catch de-staling this — a wrong event name in the agent's own skill doc is worse than a wrong name in user docs. 👍

@initializ-mk
initializ-mk merged commit 94a1e3d into main Aug 21, 2026
10 checks passed
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