Skip to content

fix(memory): correct PR citations in Vera + Riven MEMORY.md indexes#3526

Merged
AceHack merged 1 commit into
mainfrom
fix/memory-pr-citation-off-by-one-otto-cli-2026-05-15
May 15, 2026
Merged

fix(memory): correct PR citations in Vera + Riven MEMORY.md indexes#3526
AceHack merged 1 commit into
mainfrom
fix/memory-pr-citation-off-by-one-otto-cli-2026-05-15

Conversation

@AceHack
Copy link
Copy Markdown
Member

@AceHack AceHack commented May 15, 2026

Summary

Self-caught + self-corrected off-by-one PR citations in the persona MEMORY.md files I authored yesterday.

File Was Should be Why wrong
`memory/persona/vera/MEMORY.md` PR #3515 PR #3516 #3515 is the 1524z fix-PR
`memory/persona/riven/MEMORY.md` PR #3514 PR #3513 #3514 is the Alexa migration

Verified via `gh pr view` titles:

Test plan

  • Diff is exactly 2 single-digit changes in 2 files
  • Both correct PR numbers verified against actual PR titles
  • CI green
  • Auto-merge arms

🤖 Generated with Claude Code

Two off-by-one PR-number errors authored during the §33 cascade:

- memory/persona/vera/MEMORY.md cited PR #3515 (a fix-PR for the
  1524z shard) → should cite PR #3516 (the Vera §33 migration).
- memory/persona/riven/MEMORY.md cited PR #3514 (the Alexa §33
  migration) → should cite PR #3513 (the Riven §33 migration).

Both errors verified via `gh pr view` title check:
- #3513: feat(persona-riven): §33 archive migration — 12 files
- #3514: feat(persona-alexa): §33 archive migration — 11 files
- #3515: fix(shard-1524z): correct relative-link depth + raw-output mislabel
- #3516: feat(persona-vera): §33 archive migration — 1 file + persona folder + MEMORY.md

Substrate-honest catch + correction; no behavioral content moves.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 15, 2026 17:30
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@AceHack AceHack enabled auto-merge (squash) May 15, 2026 17:31
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR corrects two persona memory index citations so the migration notes point to the actual PRs that created the Vera and Riven persona folders.

Changes:

  • Corrects Vera’s archive migration citation from PR #3515 to PR #3516.
  • Corrects Riven’s archive migration citation from PR #3514 to PR #3513.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
memory/persona/vera/MEMORY.md Updates the persona folder creation PR citation.
memory/persona/riven/MEMORY.md Updates the persona folder creation PR citation.

@AceHack AceHack merged commit 0beaee2 into main May 15, 2026
27 of 28 checks passed
@AceHack AceHack deleted the fix/memory-pr-citation-off-by-one-otto-cli-2026-05-15 branch May 15, 2026 17:32
AceHack added a commit that referenced this pull request May 15, 2026
…MORY.md off-by-one PR citations (#3528)

- Self-audit while waiting on PR #3525 CI caught 2 PR-citation bugs
- Filed PR #3526 — Vera #3515#3516, Riven #3514#3513
- Both verified via gh pr view title cross-check
- Discipline: parallel work while wait-CI is per never-be-idle

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

2 participants