bd daemon sync: 2026-01-08 14:20:07

.beads/issues.jsonl

@@ -145,6 +145,7 @@
 {"id":"bd-2c5a","title":"Investigate why test issues persist in database","description":"Test issues (bd-0do3, bd-cjxp, bd-phr2, etc.) keep appearing in ready/list output, cluttering real work. These appear to be leftover test data from test runs.\n\nNeed to investigate:\n1. Why are test issues not being cleaned up after tests?\n2. Are tests creating issues in the main database instead of test databases?\n3. Should we add better test isolation or cleanup hooks?\n4. Can we add a label/prefix to distinguish test issues from real issues?\n\nThese test issues have characteristics:\n- Empty descriptions\n- Generic titles like \"Test issue 0\", \"Bug P0\", \"Issue to reopen with reason\"\n- Created around 2025-11-07 19:00-19:07\n- Some assigned to test users like \"alice\", \"bob\", \"testuser\"","notes":"## Root Cause Analysis\n\n**Problem**: Python MCP integration tests created test issues in production `.beads/beads.db` instead of isolated test databases.\n\n**Evidence**:\n- 29 test issues created on Nov 7, 2025 at 19:00-19:07\n- Patterns: \"Bug P0\", \"Test issue X\", assignees \"alice\"/\"bob\"/\"testuser\"\n- Git commit 0e8936b shows test issues committed to .beads/beads.jsonl\n- Tests were being fixed for workspace isolation around the same time\n\n**Why It Happened**:\n1. Before commit 0e8936b, `test_client_lazy_initialization()` didn't set `BEADS_WORKING_DIR`\n2. Tests fell back to discovering `.beads/` in the project root directory\n3. Auto-sync committed test issues to production database\n\n**Resolution**:\n1. ✅ Closed 29 test pollution issues (bd-0do3, bd-cjxp, etc.)\n2. ✅ Added `failIfProductionDatabase()` guard in Go test helpers\n3. ✅ Added production pollution checks in RPC test setup\n4. ✅ Created `conftest.py` with pytest safety checks for Python tests\n5. ✅ Added `BEADS_TEST_MODE` env var to mark test execution\n6. ✅ Tests now fail fast if they detect production database usage\n\n**Prevention**:\n- All test helper functions now verify database paths are in temp directories\n- Python tests fail immediately if BEADS_DB points to production\n- BEADS_TEST_MODE flag helps identify test vs production execution\n- Clear error messages guide developers to use proper test isolation\n\n**Files Modified**:\n- cmd/bd/test_helpers_test.go - Added failIfProductionDatabase()\n- internal/rpc/rpc_test.go - Added temp directory verification\n- integrations/beads-mcp/tests/conftest.py - New file with pytest safeguards","status":"closed","priority":1,"issue_type":"bug","created_at":"2025-11-07T21:31:34.845887-08:00","updated_at":"2025-11-07T21:57:30.892086-08:00","closed_at":"2025-11-07T21:57:30.892086-08:00"}
 {"id":"bd-2c64","title":"Merge: ruby-1767142029451","description":"branch: polecat/ruby-1767142029451\ntarget: main\nsource_issue: ruby-1767142029451\nrig: beads","status":"closed","priority":2,"issue_type":"merge-request","created_at":"2025-12-30T17:00:14.136568-08:00","created_by":"beads/polecats/ruby","updated_at":"2025-12-30T18:12:30.96303-08:00","closed_at":"2025-12-30T18:11:07.842778-08:00"}
 {"id":"bd-2cvu","title":"Update AGENTS.md with Agent Mail workflow","description":"Update agent workflow section to include Agent Mail coordination as optional step.\n\nAcceptance Criteria:\n- Add Agent Mail to recommended workflow\n- Show both with/without examples\n- Update \"Multi-Agent Patterns\" section\n- Cross-reference to AGENT_MAIL.md\n\nFile: AGENTS.md (lines 468-475)","status":"closed","priority":1,"issue_type":"task","created_at":"2025-11-07T22:42:51.295729-08:00","updated_at":"2025-11-08T00:52:34.288915-08:00","closed_at":"2025-11-08T00:52:34.288915-08:00","dependencies":[{"issue_id":"bd-2cvu","depends_on_id":"bd-xzrv","type":"blocks","created_at":"2025-11-07T23:04:09.773656-08:00","created_by":"daemon"}]}
+{"id":"bd-2cwz0","title":"Review PR #16199: Update documentation link","description":"Review and verify PR #16199 (anthropics/claude-code)\n\nPR: https://github.com/anthropics/claude-code/pull/16199\nAuthor: tnspacetime\n\nFix doc link in README.\n- gh pr view 16199 --repo anthropics/claude-code\n- gh pr diff 16199 --repo anthropics/claude-code\n- If good: gh pr merge 16199 --repo anthropics/claude-code --squash","status":"open","priority":2,"issue_type":"task","created_at":"2026-01-08T14:20:06.638366-08:00","created_by":"beads/crew/emma","updated_at":"2026-01-08T14:20:06.638366-08:00"}
 {"id":"bd-2d5r","title":"Fix silent error handling in RPC response writing","description":"Marshal and write errors silently ignored in writeResponse, can send partial JSON and hang clients.\n\nLocation: internal/rpc/server_lifecycle_conn.go:228-232\n\nProblem:\n- json.Marshal error ignored - cyclic reference sends corrupt JSON\n- Write error ignored - connection closed, no indication to caller  \n- WriteByte error ignored - client hangs waiting for newline\n- Flush error ignored - partial data buffered\n\nCurrent code:\nfunc (s *Server) writeResponse(writer *bufio.Writer, resp Response) {\n    data, _ := json.Marshal(resp)  // Ignored!\n    _, _ = writer.Write(data)      // Ignored!\n    _ = writer.WriteByte('\\n')     // Ignored!\n    _ = writer.Flush()             // Ignored!\n}\n\nSolution: Return errors, handle in caller, close connection on error\n\nImpact: Client hangs waiting for response; corrupt JSON sent\n\nEffort: 1 hour","status":"closed","priority":1,"issue_type":"bug","created_at":"2025-11-16T14:51:47.002242-08:00","updated_at":"2025-11-16T15:04:00.481507-08:00","closed_at":"2025-11-16T15:04:00.481507-08:00"}
 {"id":"bd-2dm49","title":"Session ended: gt-beads-crew-grip","status":"closed","priority":2,"issue_type":"event","created_at":"2026-01-07T17:42:57.665686-08:00","created_by":"beads/crew/grip","updated_at":"2026-01-07T17:42:57.71078-08:00","closed_at":"2026-01-07T17:42:57.71078-08:00","close_reason":"auto-closed session event"}
 {"id":"bd-2dwo","title":"Remove deprecated daemon logger function","description":"In cmd/bd/daemon_logger.go:131, there's a TODO to remove a deprecated function once all callers are updated to use the new signature. Need to audit callers and clean up.\n\nFile: cmd/bd/daemon_logger.go:131","status":"closed","priority":4,"issue_type":"chore","created_at":"2025-12-28T16:32:39.513622-08:00","created_by":"stevey","updated_at":"2025-12-30T15:44:43.365401-08:00","closed_at":"2025-12-30T00:28:06.54315-08:00","close_reason":"Merged via refinery"}