docs(adr): accept ADR-0033 song-id curation workflow

[PhysShell]

What

Flip ADR-0033 (human-confirmed song-id curation workflow) from Proposed → Accepted, and — per the proposal lifecycle — retire its source proposal from live discussion to historical context so the two documents don't disagree about their own authority.

Exact head: 31a332614d17f58b985cdf5ed35cd8ffb4be5f8a.

Diff is exactly 4 files, +4/-4 (status lines only), one commit:

• docs/adr/0033-song-id-curation-workflow.md — Status: Proposed → Accepted
• docs/adr/README.md — ADR-0033 row → Accepted
• docs/proposals/song-id-curation-workflow.md — status → historical context, linking ADR-0033
• docs/proposals/README.md — the song-id index row → historical context

No wording, Decisions 1–10, proposal body, implementation gates, decisions.log.md, code, fixtures, manifests, or corpus changes. Under ADR-0009 an Accepted ADR is immutable.

What acceptance authorizes — and does not

• ✅ Implementation may begin from slice 1 only (decision/validation core).
• ⛔ Slices 2–4 each require separate independent acceptance.
• ⛔ The controlled pilot stays closed until slices 1–3 are accepted; even then it writes human-confirmed song_id only to a small copied subset.
• ⛔ Production / real-corpus / full-corpus labeling remains prohibited until the pilot is independently accepted.
• No change to ADR-0031/0032.

🤖 Generated with Claude Code

https://claude.ai/code/session_012T7SRMiXmZe5v1imtrKMPi

[coderabbitai]

📝 Walkthrough

Walkthrough

ADR-0033 is marked Accepted in both its metadata and the ADR index table.

Changes

ADR status update

Layer / File(s)	Summary
Synchronize ADR status docs/adr/0033-song-id-curation-workflow.md, docs/adr/README.md	The ADR metadata and index entry both change the status from Proposed to Accepted.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

• PhysShell/griff#172: Added ADR-0033 with its original Proposed status.

Suggested reviewers: claude

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes the ADR-0033 status change to Accepted.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch claude/accept-adr-0033

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 49584961a4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".