Auto-sync beads database
This commit is contained in:
@@ -51,6 +51,7 @@
|
||||
{"id":"bd-56","title":"Fix: RemapCollisions deletes existing issue dependencies during import collision resolution","description":"Bug in updateDependencyReferences() was deleting ALL existing issue dependencies during import with --resolve-collisions, not just dependencies from imported issues.\n\nRoot cause: Function was checking if dep.IssueID was in idMapping keys (old imported IDs), but those are also IDs of existing DB issues. Fixed to only update dependencies where IssueID is in idMapping values (new remapped IDs).\n\nDuring normal import, this is now effectively a no-op since imported dependencies haven't been added to DB yet when RemapCollisions runs.\n\nFixes GH issue #120","status":"closed","priority":0,"issue_type":"bug","created_at":"2025-10-23T10:25:00.691208-07:00","updated_at":"2025-10-23T13:07:15.643069-07:00","closed_at":"2025-10-23T10:25:19.821277-07:00"}
|
||||
{"id":"bd-57","title":"init command ignores --db flag and BEADS_DB env var","description":"The bd init command hardcodes the database path to .beads/{prefix}.db and ignores both the --db flag and BEADS_DB environment variable. This prevents users from initializing databases outside the project directory.\n\nRoot cause: cmd/bd/init.go line 71 creates a local dbPath variable instead of using the global dbPath from the flag.\n\nRelated to GitHub issue #118.","notes":"Fix completed and reviewed by oracle.\n\nChanges made:\n1. Check BEADS_DB environment variable at start of init command\n2. Use global dbPath if set via --db flag or BEADS_DB, otherwise default to .beads/{prefix}.db\n3. Use proper path comparison (filepath.Abs + filepath.Clean) instead of strings.Contains to determine if database is local\n4. Only create .beads/ directory when database will actually be stored there\n5. Ensure parent directory exists for custom database paths\n6. Added comprehensive tests including edge cases:\n - Custom path with --db flag\n - Custom path with BEADS_DB env var\n - Custom path containing \".beads\" substring (to prevent false positive)\n - Flag precedence over env var\n\nOracle review feedback implemented:\n- Replaced strings.Contains heuristic with canonical path comparison\n- Added tests for edge cases\n- All init tests passing\n\nThis properly fixes GitHub issue #118.","status":"closed","priority":1,"issue_type":"bug","created_at":"2025-10-23T10:39:34.507356-07:00","updated_at":"2025-10-23T13:07:15.643266-07:00","closed_at":"2025-10-23T10:42:35.761215-07:00"}
|
||||
{"id":"bd-58","title":"MCP dep tool uses confusing parameter names (from_id/to_id instead of issue_id/depends_on_id)","description":"GitHub issue #113 reports that Claude Code is creating dependencies backwards because the MCP tool parameters are confusing.\n\nCurrent: from_id, to_id (ambiguous - does from_id block to_id or vice versa?)\nShould be: issue_id, depends_on_id (clear - issue_id depends on depends_on_id)\n\nThe CLI uses the clear naming: `bd dep add [issue-id] [depends-on-id]`\nThe MCP tool should match this for consistency and clarity.\n\nFiles to update:\n- integrations/beads-mcp/src/beads_mcp/tools.py (parameter names and docs)\n- integrations/beads-mcp/src/beads_mcp/bd_client.py (if needed)\n- integrations/beads-mcp/src/beads_mcp/bd_daemon_client.py (if needed)\n- All tests that use add_dependency","status":"closed","priority":1,"issue_type":"bug","created_at":"2025-10-23T11:04:27.792472-07:00","updated_at":"2025-10-23T13:07:15.64348-07:00","closed_at":"2025-10-23T11:09:18.893724-07:00"}
|
||||
{"id":"bd-59","title":"Document Claude Code MCP auto-approval options for beads tools","description":"Users report needing to manually approve every beads MCP command, disrupting workflow. Investigation reveals Claude Code supports MCP server-level auto-approval but NOT per-tool granularity.\n\n**Key Findings:**\n- Claude Code settings.json supports `enabledMcpjsonServers` for server-level auto-approval\n- No mechanism exists to selectively approve specific tools within an MCP server\n- Cannot auto-approve read-only tools (bd ready, bd show) while requiring confirmation for mutations (bd create, bd update)\n\n**Current Solutions:**\n1. Add `\"enabledMcpjsonServers\": [\"beads\"]` to auto-approve ALL beads tools\n2. Use `\"enableAllProjectMcpServers\": true` for project-defined servers\n3. Live with manual approvals for safety\n\n**Documentation Needed:**\n- Update PLUGIN.md with auto-approval configuration examples\n- Document trade-offs between convenience vs safety\n- Provide recommended settings for different use cases","design":"Add new section to PLUGIN.md titled \"Auto-Approval Configuration\" covering:\n\n1. Why manual approval happens (security model)\n2. Server-level auto-approval with enabledMcpjsonServers\n3. Project-level auto-approval with enableAllProjectMcpServers \n4. Trade-offs and recommendations\n5. Future: potential feature request for per-tool approval to Anthropic\n\nExample configurations for different risk profiles:\n- Conservative: Manual approval (default)\n- Balanced: Auto-approve for trusted projects only\n- Aggressive: Auto-approve all beads commands","acceptance_criteria":"- PLUGIN.md has \"Auto-Approval Configuration\" section\n- Documents enabledMcpjsonServers usage\n- Documents enableAllProjectMcpServers usage\n- Explains security trade-offs clearly\n- Provides recommended configurations\n- Links to Claude Code docs for reference","status":"closed","priority":1,"issue_type":"task","created_at":"2025-10-23T13:24:31.823818-07:00","updated_at":"2025-10-23T13:26:23.619954-07:00","closed_at":"2025-10-23T13:26:23.619954-07:00"}
|
||||
{"id":"bd-6","title":"Fix flaky CI tests in compactor and daemon modules","description":"Multiple test failures observed in CI:\n- TestCompactTier1_DryRun \n- TestCompactTier1Batch_DryRun\n- TestCompactTier1Batch_WithIneligible\n- TestMockAPI_CompactTier1\n- TestBatchOperations_ErrorHandling\n- TestCommentOperationsViaRPC\n- TestMemoryPressureDetection\n- TestPing\n\nAll failures related to 'issue has open dependents or not closed long enough' eligibility checks.\n\nSee CI run: https://github.com/steveyegge/beads/actions/runs/18688772658","status":"open","priority":1,"issue_type":"bug","created_at":"2025-10-21T23:00:08.617147-07:00","updated_at":"2025-10-23T13:07:15.643685-07:00"}
|
||||
{"id":"bd-7","title":"Compact command fails with daemon - requires --no-daemon workaround","description":"The 'bd compact' command fails with 'Error: compact requires SQLite storage' when used with the daemon (default mode), but works correctly with the '--no-daemon' flag.\n\nThe daemon RPC interface doesn't properly expose the compact command, even though the daemon itself uses SQLite storage.\n\nReproduction:\n1. Ensure daemon is running (bd daemon status)\n2. Run: bd compact --stats\n Result: Error: compact requires SQLite storage\n3. Run: bd compact --stats --no-daemon\n Result: Works correctly, shows statistics\n\nExpected behavior:\nThe compact command should work through the daemon RPC interface just like other commands (list, create, update, delete, renumber, etc.)\n\nImpact:\nUsers cannot use compact operations in the normal workflow. They must use --no-daemon which bypasses the daemon entirely.\n\nSuggested fix:\nAdd compact operation support to the daemon RPC interface, similar to how renumber and other operations are exposed.","status":"in_progress","priority":1,"issue_type":"bug","created_at":"2025-10-21T23:00:08.617147-07:00","updated_at":"2025-10-23T13:07:15.643872-07:00"}
|
||||
{"id":"bd-8","title":"Investigate stress test database pollution (vc-248)","description":"Investigation of stress tests polluting production database with 1,600+ test issues on Oct 21 at 20:24-20:25. Root cause analysis completed. Tests now verified to work correctly with proper isolation.","notes":"Bug confirmed! Tests DO pollute production DB. 1,000 test issues created at 20:46:01-20:46:02 during TestStressNoUniqueConstraintViolations. Root cause: test goroutines connect to production daemon at .beads/bd.sock instead of test daemon.","status":"open","priority":1,"issue_type":"task","created_at":"2025-10-21T23:00:08.617147-07:00","updated_at":"2025-10-23T13:07:15.644082-07:00"}
|
||||
|
||||
Reference in New Issue
Block a user