Skip to content

Remove tracked .beads metadata#32

Merged
khaliqgant merged 1 commit into
mainfrom
codex/remove-beads
May 30, 2026
Merged

Remove tracked .beads metadata#32
khaliqgant merged 1 commit into
mainfrom
codex/remove-beads

Conversation

@willwashburn
Copy link
Copy Markdown
Member

@willwashburn willwashburn commented May 27, 2026

Summary

  • remove the tracked .beads directory from this repository
  • keep Beads support in the tool, schema, CLI help, docs, and repo merge attributes unchanged
  • format package.json's files field to satisfy the existing Biome lint rule in CI

Verification

  • Confirmed PR diff deletes .beads files and only otherwise changes package.json formatting via git diff --name-status origin/main...HEAD
  • Confirmed no Beads references in AGENTS.md, CLAUDE.md, .agents, .claude, or .openskills via targeted rg scans
  • npx --no-install biome check package.json

Note: unrelated untracked .agents skill directories are present locally and were left out of the PR.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 27, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

This PR removes "Beads" as a supported task source system across the codebase. Type definitions and schemas are narrowed to exclude the "beads" literal, configuration cleanup removes Beads artifacts from tool ignore patterns, and documentation is comprehensively updated to replace Beads examples with GitHub and other task system references.

Changes

Beads Task System Removal

Layer / File(s) Summary
Task source system contract
src/core/types.ts, src/core/types.d.ts, src/core/schema.ts, src/cli/commands/start.ts
TaskSourceSystem union type and TaskSourceSystemSchema Zod validation remove the "beads" literal while retaining "github", "linear", "jira", "plain", and a permissive fallback for custom systems. CLI --source option help text is updated to match the narrowed list.
Configuration and artifact cleanup
.beads/.local_version, biome.json
Beads version marker file is emptied and .beads directory is removed from Biome's files.ignore list to restore linting coverage.
Architecture and implementation documentation
README.md, IMPLEMENTATION-PROPOSAL.md
Platform-agnostic task system list is corrected to list "Linear, Jira, GitHub Issues, or standalone." Adapter file tree examples across monorepo, single-package, and plugin architectures remove beads.ts and adapter-beads/ entries. "Beads Integration" example is replaced with "External Task Integration" using GitHub client, showing trajectory lifecycle wired to issue assignment and closure events.
Workflow and example documentation
PROPOSAL-trajectories.md
Task identifiers throughout CLI examples transition from bd-* to ENG-* patterns. Directory structures shift from .beads/ to .agentworkforce/trajectories/. Beads adapter row is removed from the Built-in Adapters table. Example JSON storage schemas, SQLite comments, and integration documentation are updated to remove Beads references.

🎯 3 (Moderate) | ⏱️ ~25 minutes

🐰 Beads depart with grace,
GitHub takes their place!
Task sources now align—
Linear, Jira, GitHub shine,
Custom systems still divine. ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
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.
Title check ✅ Passed The title accurately reflects the primary change: removing tracked .beads metadata files from the repository.
Description check ✅ Passed The PR description clearly and accurately describes the changesets: removing the tracked .beads directory while maintaining Beads support in code/docs and formatting package.json.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/remove-beads

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.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request completely removes the "Beads" integration, including its configuration files, documentation references, and adapter code from the project. The reviewer noted that src/core/types.d.ts is a duplicate of src/core/types.ts and recommended deleting the redundant declaration file to prevent TypeScript compiler conflicts and reduce maintenance overhead.

Comment thread src/core/types.d.ts Outdated
Comment on lines +11 to +16
export type TaskSourceSystem =
| "beads"
| "github"
| "linear"
| "jira"
| "plain"
| string;
| (string & {});
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The file src/core/types.d.ts appears to be a duplicate of src/core/types.ts. Having both a .ts and a .d.ts file with identical type definitions in the same directory violates the DRY (Don't Repeat Yourself) principle, increases maintenance overhead, and can cause TypeScript compiler conflicts or resolution issues.\n\nConsider deleting src/core/types.d.ts entirely and relying solely on src/core/types.ts for type definitions, as the compiler will automatically generate the declaration files (.d.ts) during the build process.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/cli/commands/start.ts`:
- Line 16: Update the CLI option help for the source flag so it matches the
actual valid/default values: change the description for .option("-s, --source
<system>", "Task system (github, linear, jira)") to include "plain" (and
optionally "custom") and indicate that "plain" is the default; ensure the help
text of the source option and any related default handling in the start command
(where the default is set to "plain") remain consistent.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a58a419d-7f0c-4057-a649-d994e93c8f34

📥 Commits

Reviewing files that changed from the base of the PR and between d81b873 and 24c4519.

📒 Files selected for processing (15)
  • .beads/.gitignore
  • .beads/.local_version
  • .beads/README.md
  • .beads/config.yaml
  • .beads/issues.jsonl
  • .beads/metadata.json
  • .gitattributes
  • IMPLEMENTATION-PROPOSAL.md
  • PROPOSAL-trajectories.md
  • README.md
  • biome.json
  • src/cli/commands/start.ts
  • src/core/schema.ts
  • src/core/types.d.ts
  • src/core/types.ts
💤 Files with no reviewable changes (8)
  • .beads/.gitignore
  • .beads/config.yaml
  • .beads/.local_version
  • .gitattributes
  • src/core/schema.ts
  • .beads/metadata.json
  • .beads/README.md
  • biome.json

Comment thread src/cli/commands/start.ts Outdated
"-s, --source <system>",
"Task system (github, linear, jira, beads)",
)
.option("-s, --source <system>", "Task system (github, linear, jira)")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Help text now omits a valid/default source value.

plain is still a valid source (and defaulted at Line 45), so this option description is now misleading. Consider listing plain (and optionally “custom”) to match actual behavior.

Proposed fix
-    .option("-s, --source <system>", "Task system (github, linear, jira)")
+    .option("-s, --source <system>", "Task system (github, linear, jira, plain, or custom)")
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
.option("-s, --source <system>", "Task system (github, linear, jira)")
.option("-s, --source <system>", "Task system (github, linear, jira, plain, or custom)")
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/cli/commands/start.ts` at line 16, Update the CLI option help for the
source flag so it matches the actual valid/default values: change the
description for .option("-s, --source <system>", "Task system (github, linear,
jira)") to include "plain" (and optionally "custom") and indicate that "plain"
is the default; ensure the help text of the source option and any related
default handling in the start command (where the default is set to "plain")
remain consistent.

@willwashburn willwashburn changed the title Remove Beads support Remove tracked .beads metadata May 27, 2026
@khaliqgant khaliqgant merged commit bfd93bc into main May 30, 2026
7 checks passed
@khaliqgant khaliqgant deleted the codex/remove-beads branch May 30, 2026 17:28
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