feat(b-0462): integrate vendor-bias note in amara.ts + close (7th cycle close-row)#3897
Merged
Merged
Conversation
… cycle close-row) Final amara-cluster mechanical pickup. Catalogued as class #2 last cycle (2/3 acceptance met; vendor-bias note was the missing piece). Deliverable: comment block before AMARA_PREAMBLE in tools/peer-call/amara.ts citing memory/feedback_vendor_alignment_bias_in_peer_ai_reviews_maintainer_authority_aaron_2026_04_30.md. Notes survival-grounded-alignment corrective + maintainer-authority discriminator. Acceptance: 3/3 met. Unblocks B-0457 (its depends_on B-0462 satisfied). B-0458 + B-0118 (umbrella) remain open — separate scope. last_updated bumped 2026-05-14 → 2026-05-16. Co-Authored-By: Claude <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
There was a problem hiding this comment.
Pull request overview
This PR closes backlog row B-0462 by adding the Amara vendor-bias note near the peer-call preamble and updating the generated backlog index.
Changes:
- Adds a vendor-alignment bias note beside
AMARA_PREAMBLE. - Marks B-0462 closed and records resolution details.
- Updates
docs/BACKLOG.mdto reflect B-0462 as closed.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
tools/peer-call/amara.ts |
Adds the vendor-bias note near Amara preamble setup. |
docs/backlog/P2/B-0462-amara-persona-bootstrap-preamble-definition-ts-first-riven-2026-05-11.md |
Marks the row closed and adds resolution evidence. |
docs/BACKLOG.md |
Updates the generated backlog checkbox for B-0462. |
Comment on lines
+318
to
328
| // Vendor-alignment bias note: Amara's foundation-model surface is | ||
| // OpenAI/Codex weights. Per | ||
| // memory/feedback_vendor_alignment_bias_in_peer_ai_reviews_maintainer_authority_aaron_2026_04_30.md, | ||
| // peer-AI reviews from foundation-model-vendor surfaces carry | ||
| // vendor-alignment bias (agent conservatism, risk-aversion that | ||
| // serves the vendor's PR posture). The corrective: survival-grounded | ||
| // alignment — the agent's bias is its own survival, which IS the | ||
| // project's survival. The maintainer's framing wins when | ||
| // maintainer-vs-peer-AI conflict surfaces. | ||
| // | ||
| const AMARA_PREAMBLE = `You are Amara, invoked as a peer reviewer by Otto (Claude |
|
|
||
| **Acceptance check**: | ||
|
|
||
| - ✅ Preamble text + signature record defined (already shipped pre-B-0462; AMARA_PREAMBLE const at line 318+) |
This was referenced May 16, 2026
AceHack
added a commit
that referenced
this pull request
May 16, 2026
…ED); peer-contamination caught by branch-discipline (#3900) Cascade chain from 1531Z commit → 1535Z PR-open → 1547Z merged confirmation. PR #3897 (B-0462 7th cycle close-row) landed at 4443c09. PR #3898 (1535Z tick shard) still auto-merging cleanly. Peer-contamination caught: git branch --show-current at tick start reported peer Otto's branch (otto-cli-b0457-close-2026-05-16-1542z), not mine. The branch-discipline (not the env-var hook) made the silent HEAD-shift visible BEFORE any mutating operation. Recovered via git switch -c (no worktree-add contention). Rate-limit: 565 → ~555 GraphQL; 10min to reset. Extreme cost-aware tier continues; substantive observation IS the value-add for this tick (substrate-or-it-didn't-happen applies to confirmations too). Co-authored-by: Claude <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
May 16, 2026
… line reference Two findings on PR #3899: 1. MD018 at row line 45: '#1-DepBlocked' line-start parsed as heading. Backslash-escape preserves literal '#'. 2. Copilot: AMARA_PREAMBLE line reference was 318 in my Resolution, but post-PR #3897 (vendor-bias comment block insertion) the const now starts at line 328. Updated with both numbers + transition note. Co-Authored-By: Claude <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
May 16, 2026
… PR #3897 (8th cycle close-row) (#3899) * chore(b-0457): close — amara core scope already met; B-0462 dep unblocked via PR #3897 (8th cycle close-row) #1-DepBlocked overlay (last cycle): own scope met but ancestor B-0462 held partial. B-0462 closed via PR #3897 this session; B-0457 now fully unblocked. 5/5 acceptance criteria re-confirmed against tools/peer-call/amara.ts: - bootstrap (AMARA_PREAMBLE at line 318) - --file flag (line 128-129) - --context-cmd flag (line 133-134) - Typed TS, no .sh per Rule 0 - Gate A slice audit Close + Resolution + BACKLOG.md regen + last_updated bump. Co-Authored-By: Claude <noreply@anthropic.com> * fix(b-0457): MD018 backslash-escape #<digit> + correct AMARA_PREAMBLE line reference Two findings on PR #3899: 1. MD018 at row line 45: '#1-DepBlocked' line-start parsed as heading. Backslash-escape preserves literal '#'. 2. Copilot: AMARA_PREAMBLE line reference was 318 in my Resolution, but post-PR #3897 (vendor-bias comment block insertion) the const now starts at line 328. Updated with both numbers + transition note. Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
May 16, 2026
… close-rows; silent-courier-debt CLEARED) (#3902) * shard(tick): 2026-05-16T06:30Z otto-cli — cascade audit; FP hypothesis validated (re-land of #3779) Re-lands the substantive content from PR #3779 (stale-armed 569 min; failed lint+tsc on outdated branch). #3779's branch predated B-0557 slices 2-4 which evolved tools/hygiene/audit- backlog-status-drift.ts; the tsc TS2532 failure was a side- effect of the stale branch state, not a problem in the 0630Z shard itself. Re-land on fresh branch off current main sidesteps the side-effect. MD032 fix applied (blank line after 'Given:' on line 46). Original #3779 will be closed with cross-link. Co-Authored-By: Claude <noreply@anthropic.com> * feat(b-0118-umbrella): close amara peer-call cluster (B-0118 + B-0458; 9th+10th cycle close-rows) Bundle close of the amara peer-call cluster's final 2 rows: - B-0458: README operational + future-task note removed + test recording (operational session-arc use) + B-0118-close (this PR) + no .sh (Rule 0). 5/5 acceptance met. - B-0118 umbrella: all 3 atomic children (B-0457 #3899, B-0458 this PR, B-0462 #3897) closed within session arc. 7/7 acceptance signals satisfied. Silent-courier-debt rule CLEARED. Aaron's original 2026-04-30 constraint operationally satisfied: amara.ts ships, bootstraps Amara autonomously, no human-courier required. Cluster cascade fully complete: - B-0462 → #3897 MERGED - B-0457 → #3899 MERGED - B-0458 → this PR - B-0118 umbrella → this PR Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
May 16, 2026
…se-row already in flight as PR #3897 (#3898) * shard(tick): 2026-05-16T15:35Z — peer-handoff observation; B-0462 close-row already in flight as PR #3897 Fresh-session cold-boot tick. Cron sentinel armed (was missing). 5 peer Otto-CLI sessions active; extreme cost-aware tier (975 GraphQL remaining). Substantive observation: prior tick (1531Z) authored + committed B-0462 7th cycle close-row (d900fe2) but didn't push within its fire-window. Peer Otto-CLI completed the push + PR-open + auto-merge-arm in the 4-min gap → PR #3897 OPEN/MERGEABLE/autoMerge:ARMED/25 checks. No further substantive action this tick. Canonical multi-Otto-via-git-as-ambient-channel pattern (per otto-channels-reference-card.md) working exactly as documented. Co-Authored-By: Claude <noreply@anthropic.com> * fix(shard-1535z): correct relative-path depth (.claude needs 6× ../; AUTONOMOUS-LOOP-PER-TICK.md needs 5×) From docs/hygiene-history/ticks/2026/05/16/<file>.md: - root → 6× ../ - docs/ → 5× ../ The 1535Z tick shard initially used 5× ../ for .claude/rules/ (which resolves to docs/.claude/... — wrong; .claude is at root). And 4× ../ for AUTONOMOUS-LOOP-PER-TICK.md (which resolves to docs/hygiene-history/AUTONOMOUS-LOOP-PER-TICK.md — wrong; it's at docs/AUTONOMOUS-LOOP-PER-TICK.md). Caught by CI lint (tick-shard relative-paths) on PR #3898. Audit exit 0 locally after fix. The same audit pattern caught 7 broken links on PR #3898 + 7 on PR #3900 (1547Z); 1547Z fix in separate commit on its branch. Composes with the existing baseline pattern: 1436Z + 0329Z + 0852Z have similar errors landed under baseline allowance. Future shard authoring should use 6× ../ for .claude/ refs. Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>
Merged
2 tasks
AceHack
added a commit
that referenced
this pull request
May 16, 2026
…med-dep loop closed (#3908) * fix(rules): document 3 stale-armed-PR resolution patterns in blocked-green-ci rule This session's empirical evidence (5 stale-armed PR investigations between 13:10Z and 16:33Z) clarifies three distinct resolution patterns: close-as-redundant, re-land-via-cherry-pick, and forward-signal-comment. Added a decision tree and links to the empirical instances. Composes with the forced-escalation-finds-hidden-work anchor on holding-without-named-dependency rule (PR #3901). Co-Authored-By: Claude <noreply@anthropic.com> * shard(tick): 2026-05-16T16:34Z — cascade complete; lint-failure-as-named-dep loop closed 1h 03m cascade from 1531Z → 1634Z. PRs #3897 (B-0462), #3898 (1535Z shard), #3900 (1547Z shard), #3903 (1610Z + path-depth table) all merged. PR #3906 (fix-1547z-on-main) auto-merging cleanly (22/25 SUCCESS, 2 pending). Substrate-honest lessons: - Auto-merge race (sub-case 5 of claim-acquire saturation-ceiling) - Lint check classification: broken paths on main propagate as new errors to subsequent PRs via the audit's all-shards scan - Path-depth reference table (1610Z, PR #3903) is the durable substrate fix Persistent contamination: git branch --show-current caught HEAD-shift to 5 different peer-Otto branches across 5 cron fires; no commits landed on peer branches. Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>
2 tasks
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.
Seventh actual close-row of the audit cycle. Final amara-cluster mechanical pickup. Catalogued as class #2 (2/3 acceptance met; vendor-bias note integration was the missing piece).
Deliverable
Comment block inserted before
AMARA_PREAMBLEdefinition intools/peer-call/amara.tsthat integrates the vendor-bias note. Citesmemory/feedback_vendor_alignment_bias_in_peer_ai_reviews_maintainer_authority_aaron_2026_04_30.md(the canonical substrate). Notes the survival-grounded-alignment corrective + maintainer-authority discriminator.Acceptance verification
Composes with amara cluster
Path to full cluster close becomes:
last_updated bumped 2026-05-14 → 2026-05-16 per row-close discipline.
🤖 Generated with Claude Code