review(pr-1247-followup): rename file to reflect both triggers + fix bare-grep placeholder

[AceHack]

Summary

Two Copilot post-merge findings on PR #1247 (already merged):

P2 — filename mismatch. MEMORY.md entry described two-trigger discipline (creation + pickup); filename was feedback_at_pickup_time_prereq_check_discipline_*, reading as pickup-only. Renamed to feedback_depends_on_backlog_search_discipline_at_creation_and_at_pickup_*.

P1 — bare grep placeholder. At-creation procedure had grep -l "tags:.*<tag>" ... where ... was a placeholder; copy-paste risk turns it into repo-wide grep pulling in references/ clones. Replaced with explicit docs/backlog/P*/B-*.md glob + warning sentence.

Why follow-up PR (not deferred)

Per Aaron 2026-05-02: "the answer to if it matters is almost always yes in real life lol" — post-merge findings on substrate-rule memos warrant their own follow-up PR rather than backlog deferral.

What changed

• git mv of the memory file (97% similarity preserved)
• MEMORY.md pointer updated to new filename
• Procedure body has explicit path glob in the grep example + anti-bare-placeholder warning sentence

Test plan

• No remaining inbound references to old filename (verified via grep -rln)
• Frontmatter name: already reflects both triggers (no change needed)
• CI green

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This follow-up PR cleans up a recently added memory artifact so its filename matches the two-trigger discipline described in the memo, and it tightens the documented backlog-search example to avoid an unsafe copy-paste pattern. It fits the repository’s memory/index maintenance workflow by keeping the memory/ corpus internally consistent and less error-prone for future edits.

Changes:

• Renames the memory file so the filename reflects both trigger points: row creation and pickup time.
• Updates memory/MEMORY.md to point at the renamed file.
• Replaces the bare grep placeholder in the memo with an explicit backlog path glob and a warning about repo-wide grep misuse.

Reviewed changes

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

File	Description
memory/feedback_depends_on_backlog_search_discipline_at_creation_and_at_pickup_aaron_2026_05_02.md	Clarifies the at-creation search procedure with an explicit docs/backlog/P*/B-*.md grep target and warning text.
memory/MEMORY.md	Updates the index entry to reference the renamed memory file.

[Copilot]

Pull request overview

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

[AceHack]

All 5 post-merge findings addressed in PR #1251:

• 3 fixed in code: line-35 stale ref in future-domain memo (the surviving ref after prior only fired on line 90); MEMORY.md count 12→13 (specialists table); MEMORY.md count 5→6 (procedure skills table)
• 2 tick-shard findings (2354Z + 2357Z) NOT corrected — tick shards are append-only history preserving agent-belief-at-time. The shards accurately reflect Otto's belief at write-time; retroactively rewriting them would be ahistorical. The follow-up PR fixes the underlying issue.

Resolving threads with cross-reference to the follow-up PR.