docs(console): record the Forgejo migration and what it left behind - #524
Merged
Conversation
The Plane -> Forgejo migration ran off direct operator requests and was never tracked in the backlog, so nothing in `.console/` tells a future session it happened or why the codebase looks the way it does now. Adds the Done entry (with the recon finding that reshaped it: Plane was never live on this host, so the planned drain was vacuous) and the two items it left open: * **CI runs ~1,830 fewer tests than exist.** No longer hypothetical — #521 merged a NEW failing test because `tests/test_dependency_check.py` is in no CI job, and main was red until #522. #513 was the same story a week earlier. Recorded with the 6 currently-failing tests that make switching the gate on a decision rather than a chore, and the sensible order: fix those 6, then add the suites in the PR that proves them green. * **`audit` on Forgejo Actions** — the last line of the PR-adapter spec, blocked on an operator installing a runner. Everything upstream is done. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Plane → Forgejo migration ran off direct operator requests and was never tracked in the backlog, so nothing in
.console/tells a future session it happened or why the codebase looks the way it does now. This records it, plus the two things it left open.Done entry
The recon finding that reshaped the whole plan is in there: Plane was never live on this host. Port 8080 belonged to an unrelated stack's
status-service, the config carried the all-zeros placeholder project id, and every board-worker cycle had been loggingfailed to list issuesfor its entire life. The "drain to zero" the board spec planned for was vacuous — the cutover was not move the board, it was give the fleet a live board for the first time.Also recorded: three lessons already in
.console/log.md, all one shape — a measurement reporting something other than the truth. The "egress flake" was my own gate scripts leakingOC_EGRESS_SNI_STRICT=1into pytest (that suite was always green). Worktrees lack the venv, the env file and the editable install, so barepython/tythere measure the main checkout. And an empty directory left bygit rmis still an importable PEP 420 namespace package.Up Next: CI runs ~1,830 fewer tests than exist
No longer hypothetical. #521 merged a new failing test because
tests/test_dependency_check.pyis in no CI job — CI was green, the council verdict was SUCCESS, main was red until #522. #513 was the same story a week earlier: an #509 regression sat unnoticed intests/maintenance/.Adding the suites is a one-line workflow change. What makes it a decision is that 6 tests in them are currently failing, so turning the gate on blocks every merge until they're fixed. The entry lists all 6 by name and proposes the sensible order: fix them, then add the suites in the PR that proves them green.
Up Next:
auditon Forgejo ActionsThe last line of the PR-adapter spec, blocked on an operator installing a
forgejo-runner. Everything upstream is done —ForgejoPRClientis verified against the live instance, andpr_backend(defaultgithub) is the single switch that performs the cutover.Verification
The entry's central claim is self-checked: the un-gated suites fail on exactly 6 tests, confirmed in this branch. Custodian audit clean, census OK, backlog at 32% of the OC2 500K limit.
🤖 Generated with Claude Code