fix(otto-279): role-refs not persona-names in docs/ surfaces (3 hits)#3572
Merged
Conversation
Follow-up to PR #3570 (.claude/ surfaces). Same Otto-279 fix pattern applied to next-batch docs/ files: - docs/GITHUB-SETTINGS.md:260 (B-0155 Phase 1 audit attribution) - docs/POST-SETUP-SCRIPT-STACK.md:87 (bash-forever decision attribution) - docs/launch/2026-05-13-otto-claude-desktop-bootstream-tight.md:89 ("the point" attribution) Pattern: `Per Aaron 2026-MM-DD: "quote"` → `Per the human maintainer 2026-MM-DD: "quote"`. Verbatim quote content preserved. Per Otto-279 history-surface carve-out (docs/AGENT-BEST-PRACTICES.md): docs/* outside the closed attribution list (memory/, docs/research/, docs/aurora/, docs/ROUND-HISTORY.md, docs/DECISIONS/, docs/hygiene-history/, docs/pr-preservation/, commit messages) uses role-refs, not persona names. Auditor verification: `bun tools/hygiene/audit-orphan-role-refs.ts` re-run shows 0 `Per Aaron` hits in docs/* live surface after fix. Remaining hit `docs/trajectories/typescript-bun-migration/slice-audits.md:759` is an `orphan-ferry-ref` class (different fix pattern); deferred to follow-up. Composes with: PR #3570 (.claude/ batch), tools/hygiene/audit-orphan-role-refs.ts, Otto-279 carve-out, [`.claude/rules/honor-those-that-came-before.md`](.claude/rules/honor-those-that-came-before.md). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This PR applies Otto-279 role-reference cleanup to selected documentation surfaces by replacing direct maintainer name attribution with role-based attribution.
Changes:
- Updates attribution wording in GitHub settings documentation.
- Updates attribution wording in post-setup script stack documentation.
- Updates attribution wording in one launch-substrate document.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
docs/GITHUB-SETTINGS.md |
Replaces direct maintainer attribution with role-reference wording. |
docs/POST-SETUP-SCRIPT-STACK.md |
Replaces direct maintainer attribution while preserving the quoted decision text. |
docs/launch/2026-05-13-otto-claude-desktop-bootstream-tight.md |
Replaces attribution in a launch-substrate history surface. |
3 tasks
AceHack
added a commit
that referenced
this pull request
May 15, 2026
…aces (3 hits) (#3573) - All prior-tick PRs merged: #3567 (B-0532), #3569 + #3571 (shards), #3570 (.claude/ batch) - PR #3572 — 3 docs/ Per Aaron hits fixed; same pattern as .claude/ batch - Audit-first-then-decide pattern continues from 1959Z - Otto-279 cleanup: 6 fixes across 2 ticks; remaining batches deferred - 18-tick session arc Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
4 tasks
docs/launch/** is a history/launch-substrate surface per AGENT-BEST-PRACTICES.md:709-721 and copilot-instructions.md:337. Named attribution (Aaron) must be preserved verbatim; role-ref replacement is only for instructional/operational surfaces. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Member
Author
|
Fixed in 6e24876 — reverted |
AceHack
added a commit
that referenced
this pull request
May 15, 2026
…ixes) (#3575) - PR #3572 + #3573 wait-CI - PR #3574 — 9 Per Codex hits in tools/hygiene/audit-*.ts - Cleanup cadence: ".claude/ + docs/ + tools/" = 3 batches, 15 fixes across 3 ticks - Each batch ~1 surface, 3-9 line-edits, mechanical - Remaining: 1 Per Aaron in test file, 1 Per Copilot in tools/peer-call, orphan-ferry-refs - 19-tick session arc Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
4 tasks
AceHack
added a commit
that referenced
this pull request
May 15, 2026
…ts) (#3576) Otto-279 cleanup batch #4. Same fix pattern as batches #1-#3 (PRs #3570, #3572, #3574). Files: - tools/github/poll-pr-gate-batch.test.ts:9 — `Per Aaron 2026-05-01` → `Per the human maintainer 2026-05-01` - tools/peer-call/append-identity-receipt.ts:307 — `Per Copilot 2026-05-06 review on PR #1702` → `Per PR #1702 review 2026-05-06` Verbatim quote content preserved in both cases. Auditor verification: `bun tools/hygiene/audit-orphan-role-refs.ts` re-run shows 0 hits in target scope after fix. Remaining (next-batch candidates): - src/Core/Maji.fs + tests + tools/hygiene/audit-agencysignature-main-tip.ts + tools/hygiene/validate-agencysignature-pr-body.ts + docs/trajectories/.../slice-audits.md — `orphan-ferry-ref` class (different fix pattern — requires per-occurrence judgment) Composes with: PR #3570 (.claude/), PR #3572 (docs/), PR #3574 (tools/hygiene/audit-*.ts), tools/hygiene/audit-orphan-role-refs.ts, Otto-279 carve-out. Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
May 15, 2026
…tools/github/test + tools/peer-call (#3577) - PR #3572 + #3574 + #3575 wait-CI - PR #3576 — 2 final hits (Per Aaron in test, Per Copilot in peer-call) - 17 per-name-attribution fixes across 4 batches; all audit-flagged hits addressed - Remaining: orphan-ferry-ref class (different fix pattern; per-occurrence judgment) - Observation: audit regex doesn't catch multi-name attributions (e.g., "Per Codex + Copilot") - 20-tick session arc Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
May 15, 2026
…ilename paths (#3578) Captures the substrate-honest residual after this session's 4-batch Otto-279 per-name-attribution cleanup arc completed. Two issues: Part A — orphan-ferry-ref cleanup (5+ hits) requires per-occurrence judgment (not mechanical substitution), so deferred to its own row. Part B — audit-orphan-role-refs.ts has a false-positive: filenames containing "ferry-N" (e.g., maji-formal-operational-model-deep-research- peer-courier-ferry-2026-04-26.md) get flagged when referenced inside .fs/.ts/.md files. The audit regex needs to skip filename-path matches. ID allocation discipline: B-0531/B-0532/B-0533/B-0534/B-0535 taken on main; B-0536 verified free via gh pr list search. Composes with: 4-batch Otto-279 arc this session (PRs #3570, #3572, #3574, #3576), memory/feedback_orphan_role_ref_after_name_stripping_*.md (fix-pattern reference), tools/hygiene/audit-orphan-role-refs.ts. Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
May 15, 2026
…s (9 hits) (#3574) * fix(otto-279): role-refs not persona-names in tools/hygiene/audit-*.ts (9 hits) Follow-up batch #3 to PR #3570 (.claude/ surfaces) and PR #3572 (docs/ surfaces). Same Otto-279 fix pattern applied to tools/hygiene/audit-*.ts code comments. Files: - tools/hygiene/audit-backlog-items.ts:505 (1 hit) - tools/hygiene/audit-lost-files.ts:103, 145, 166, 313 (4 hits) - tools/hygiene/audit-promotion-ledger.ts:551 (1 hit) - tools/hygiene/audit-trajectories.ts:204, 271, 363, 369 (4 hits — 3 flagged by audit + 1 line-wrapped variant audit didn't catch via regex) Pattern: `Per Codex 2026-05-06 review[ on PR #1702]: "..."` → `Per [PR #1702 ]review 2026-05-06: "..."`. Drops persona/tool name attribution; preserves PR-thread reference + temporal anchor. Per Otto-279 carve-out: "Per <Name> 2026-MM-DD" belongs on history surfaces only. Code-comment review attributions in tools/* count as code-surface, not history surface. Auditor verification: `bun tools/hygiene/audit-orphan-role-refs.ts` re-run shows 0 `Per Codex` hits in tools/hygiene/audit-*.ts after fix. Remaining (next-batch candidates): - tools/github/poll-pr-gate-batch.test.ts:9 `Per Aaron` (test comment) - tools/peer-call/append-identity-receipt.ts:307 `Per Copilot` (sibling class — Copilot is also tool-name-as-persona) - src/Core/Maji.fs + tests (orphan-ferry-refs — different fix pattern) Composes with: PR #3570 (.claude/ batch), PR #3572 (docs/ batch), tools/hygiene/audit-orphan-role-refs.ts (the detection tool), Otto-279 carve-out. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * fix(otto-279): add PR #1702 number to ambiguous attribution comments (2 hits) Copilot thread PRRT_kwDOSF9kNM6Cd-oR: 'Per PR-review 2026-05-06' lacked the concrete PR number used elsewhere in the file. Normalised to 'Per PR #1702 review 2026-05-06' on lines 103 and 166 so grep-ability and traceability are consistent across all four occurrences. Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
May 15, 2026
…fix / row / quiet (#3583) Captures the explicit pattern empirically validated across the 23-tick Otto-CLI session 2026-05-15 (1718Z–2038Z): when uncertain what to do at a tick boundary, run an audit FIRST; let the result determine which of three actions applies. Three-way classification: - Mechanical fix-class → fix this tick as small atomic PR (Otto-279 4-batch arc: 17 fixes, PRs #3570/#3572/#3574/#3576) - Content-judgment class → file as backlog row (B-0535, B-0536, B-0537 — substrate-honest residual capture) - Null actionable → write quiet checkpoint shard (1919Z, 1952Z, 2038Z precedents) Resolves apparent tension between never-be-idle (do speculative work), razor-discipline (don't manufacture findings), and holding-without- named-dependency-is-standing-by-failure (Holding without real wait is the failure pattern). The audit output routes the decision. Composes with razor-discipline, no-op-cadence-failure-mode, holding- without-named-dependency-is-standing-by-failure, never-be-idle, encoding-rules-without-mechanizing, substrate-or-it-didn't-happen. Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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
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.
Summary
Follow-up batch to PR #3570 (.claude/ surfaces). Same Otto-279 fix pattern applied to next-batch docs/ surfaces.
Files
Auditor verification
`bun tools/hygiene/audit-orphan-role-refs.ts` after fix: 0 `Per Aaron` hits in docs/* live surface.
Out of scope (next-batch candidates)
Test plan
🤖 Generated with Claude Code