dries/ocman

Fork 1

chore: drop stale Claude Code references #485

Open

dries wants to merge 3 commits from chore/drop-claudecode-refs into main

dries commented

2026-08-01 00:43:29 +02:00

Owner

Summary

The internal/platforms/claudecode/ adapter no longer exists, but 45-odd comments, doc lines, a Grafana panel description and two test names still referenced it — including the project-structure listing in the contributing docs.
Reworded them to platform-neutral phrasing, or deleted the line where it only described the removed adapter.
spec/ left untouched: those are historical design records.

Testing

go build ./... && go vet ./...
go test ./internal/platforms/... ./internal/server/... ./internal/state/...
tsc -b, vitest on the two touched test files

## Summary - The `internal/platforms/claudecode/` adapter no longer exists, but 45-odd comments, doc lines, a Grafana panel description and two test names still referenced it — including the project-structure listing in the contributing docs. - Reworded them to platform-neutral phrasing, or deleted the line where it only described the removed adapter. - `spec/` left untouched: those are historical design records. ## Testing - `go build ./... && go vet ./...` - `go test ./internal/platforms/... ./internal/server/... ./internal/state/...` - `tsc -b`, vitest on the two touched test files

dries added 1 commit

2026-08-01 00:43:29 +02:00

chore: drop stale Claude Code references

CI / Frontend (pull_request) Successful in 6m49s

Details

CI / Backend (pull_request) Failing after 6m2s

Details

CI / Coverage Results (pull_request) Successful in 32s

Details

CI / Build (pull_request) Successful in 4m30s

Details

CI / Playwright E2E (pull_request) Successful in 8m9s

Details

CI / Semantic Tag (pull_request) Has been skipped

Details

e604f1d53c

The claudecode adapter is long gone; only comments, docs and test
names still named it.

forgejo-actions commented

2026-08-01 00:56:36 +02:00

Coverage ratchet: ✅ pass

Suite	Baseline	This PR	Δ
frontend	67.73%	67.73%	+0.00	✅
go	78.80%	79.10%	+0.30	✅

Tolerance: -0.1%. Baseline stored on gh-pages.

### Coverage ratchet: ✅ pass | Suite | Baseline | This PR | Δ | | |---|---|---|---|---| | frontend | 67.73% | 67.73% | +0.00 | ✅ | | go | 78.80% | 79.10% | +0.30 | ✅ | _Tolerance: -0.1%. Baseline stored on `gh-pages`._

dries added 1 commit

2026-08-01 01:24:33 +02:00

ci: re-measure Go coverage once before failing the ratchet

CI / Backend (pull_request) Failing after 5m16s

Details

CI / Frontend (pull_request) Successful in 7m10s

Details

CI / Coverage Results (pull_request) Successful in 32s

Details

CI / Playwright E2E (pull_request) Successful in 8m55s

Details

CI / Build (pull_request) Successful in 4m36s

Details

CI / Semantic Tag (pull_request) Has been skipped

Details

ae86478033

A loaded runner measured 77.1% against a 78.8% baseline while every
per-package number stayed within 0.1% of a passing run, and 78.9%
locally. Coverage of background goroutines depends on how much of them
ran before the binary exited, so the gate needs a second sample; a real
regression still fails both times.

dries added 1 commit

2026-08-01 02:11:49 +02:00

test: cover tmux wrappers, opencode operations, mcp split tools, workflow service

CI / Frontend (pull_request) Successful in 6m28s

Details

CI / Backend (pull_request) Successful in 7m20s

Details

CI / Coverage Results (pull_request) Successful in 35s

Details

CI / Playwright E2E (pull_request) Successful in 8m45s

Details

CI / Build (pull_request) Successful in 3m40s

Details

CI / Semantic Tag (pull_request) Successful in 5s

Details

ddf2cba8a1

Adds coverage for five untested code areas: the tmux session wrappers,
the six 0%-coverage Adapter operations, the filesystem/system handlers,
the MCP split tools and the workflow service's validation and
error paths. Go total 78.9% -> 81.3%.