Skip to content

Remove PRD link inclusion from bead conversion#282

Merged
subsy merged 1 commit intomainfrom
claude/investigate-issue-215-YUAor
Feb 4, 2026
Merged

Remove PRD link inclusion from bead conversion#282
subsy merged 1 commit intomainfrom
claude/investigate-issue-215-YUAor

Conversation

@subsy
Copy link
Owner

@subsy subsy commented Feb 4, 2026

Summary

Removed the logic that automatically includes PRD (Product Requirements Document) links as external references when converting stories to beads.

Changes

  • Deleted the conditional block that appended --external-ref prd:{prdPath} to the story arguments during bead conversion
  • This simplifies the conversion process by removing the automatic PRD reference injection

Details

The removed code was checking if a prdPath was available and, if so, inserting an external reference argument into the bead command. This functionality is no longer needed in the conversion workflow.

https://claude.ai/code/session_013iLssPBjrhK6LJG8Pqkm8n

Summary by CodeRabbit

  • Bug Fixes
    • Removed automatic external PRD reference injection from epic and story bead creation. Beads will no longer include external reference parameters during creation.

When converting PRD to beads format, the same external_ref value was
being assigned to both the epic and all child stories. This violated
the UNIQUE constraint on external_ref during bd sync.

The external_ref should only be set on the epic, as it identifies the
source PRD document for the entire feature.

Fixes #215

https://claude.ai/code/session_013iLssPBjrhK6LJG8Pqkm8n
@vercel
Copy link

vercel bot commented Feb 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
ralph-tui Ignored Ignored Feb 4, 2026 6:17pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 4, 2026

Walkthrough

This change removes external PRD reference injections from bead command invocations in the convert command. Specifically, the --external-ref prd:<path> argument insertion before --silent flags was deleted from both epic and per-story creation argument arrays.

Changes

Cohort / File(s) Summary
Convert Command PRD Reference Removal
src/commands/convert.ts
Removed --external-ref prd:<path> argument injections from both epic and per-story bead creation command invocations within the convertToBeads function.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related issues

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: removing PRD link inclusion from bead conversion logic, which matches the deleted conditional block that injected external references.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude/investigate-issue-215-YUAor

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

Comment @coderabbitai help to get the list of available commands and usage tips.

@subsy subsy merged commit 67aad2d into main Feb 4, 2026
7 checks passed
sakaman pushed a commit to sakaman/ralph-tui that referenced this pull request Feb 15, 2026
…YUAor

Remove PRD link inclusion from bead conversion
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

Comments