2026-02 Quality and Onboarding Opportunities
Status: Historical Snapshot (Partially Completed) Reviewed: 2026-02-11 Inputs: recent commits (
v0.9.7..HEAD),go test ./...,golangci-lint runFor current status, use
docs/IMPLEMENTATION_STATUS.mdandROADMAP.md.
Snapshot
go test ./...: passing (at time of review)golangci-lint run: passing after one low-risk fix ininternal/devbox/detect/hash_test.go- historical coverage at review time (
go tool cover -func): 21.2% - hook reliability improvement (2026-02-11):
loom agentsupports HUD->daemon fallback andheartbeat --ensure-sessionbootstrap for notify-only clients.
Completion Since This Review
Items from this plan that have shipped (see ROADMAP.md / CHANGELOG.md):
- Docs + command guardrails in CI (
scripts/ci/check_docs_guardrails.shand CLI help smoke checks). make bootstrap-localonboarding workflow.- Significant MCP server coverage growth and additional edge-case tests.
Items still active from this plan:
- Additional daemon lifecycle + devbox integration coverage toward the current 40% target.
- Broader migration to standardized MCP error handling across remaining legacy servers.
Priority Opportunities
1) Raise coverage on MCP servers with no direct tests
Many cmd/mcp-* servers still have no package-level tests. High-value candidates:
mcp-devbox(new, complex lifecycle and backend orchestration)mcp-agent-context(critical stateful core)mcp-release,mcp-substack,mcp-itchio(new additions)- frequently used infra servers:
mcp-git,mcp-grafana,mcp-docker,mcp-redis
Proposed baseline:
- Add one happy-path test per server.
- Add one argument validation/error-path test per server.
- Add one external-call failure mapping test (
mcperrorresponse shape).
2) Add a docs guardrail in CI
Recent docs drift happened around HUD/devbox features and command workflows.
Proposed checks:
- Add a CI job that fails if
README.md,docs/, andCHANGELOG.mdare unchanged for user-visible commits (convention/tag-based). - Add a lightweight command smoke check (
go run ./cmd/loom --help) in CI to catch doc/command drift early.
Status update (2026-02-11):
- Implemented in GitLab CI + GitHub Actions.
- Added reusable script:
scripts/ci/check_docs_guardrails.sh.
3) Improve first-run onboarding automation
Current onboarding requires users to infer sequence across multiple docs.
Proposed additions:
- Add
make bootstrap-localthat runs:make buildmake install-core./bin/loom sync all --regen --loom-mode./bin/loom check
- Print explicit next steps (
loom start,loom hud --port 3333) at the end.
4) Strengthen devbox quality gates
mcp-devbox is now a core path and should have stronger regression protection.
Proposed additions:
- Integration tests covering both Docker and K8s backend selection paths.
- Tests for mount/workdir behavior in monorepo and out-of-tree project layouts.
- Tests for async exec lifecycle cleanup (stale exec IDs, timeout behavior).
5) Standardize error-handling adoption across MCP servers
The documented standard (validate + mcperror) is clear, but adoption is uneven across older servers.
Proposed implementation:
- Add a checklist-driven migration tracker for all
cmd/mcp-*directories. - Prioritize high-traffic servers first.
- Include at least one negative test per migrated server to lock in response format.
Suggested Execution Order
- Coverage baseline for
mcp-devbox,mcp-agent-context, and newest MCP servers. bootstrap-localonboarding command.- CI docs/command smoke guardrails.
- Broader error-handling standardization sweep.