DO NOT MERGE — MX-222 negative control, proving the new gate can go red - #20
Closed
MGrin wants to merge 2 commits into
Closed
DO NOT MERGE — MX-222 negative control, proving the new gate can go red#20MGrin wants to merge 2 commits into
MGrin wants to merge 2 commits into
Conversation
The only CI job here was `managed-install`, which proves the plugin builds from a git install with runtime deps only. That is a real check and it stays. It is not a check on whether the code works: `bb plugin build` bundles without typechecking, and `npm test` ran only when somebody typed it, so a change that broke all 269 tests still got a green check. Separate job, not extra steps: managed-install installs `--omit=dev`, so typescript and @types/* are absent by the time it ends, and a build failure and a test failure should stay distinguishable at a glance. Verified locally on this commit: `npm ci` + `npm run typecheck` + `npm test` all exit 0, 269 pass / 0 fail. What it does not cover, stated in the workflow: no test imports server.ts (1678 lines), so the wiring, handlers and schedules are typechecked and never executed — the same hole MX-219 measured in bb-plugin-bus. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Owner
Author
|
Read: the Test job went red on a broken assertion while Managed install stayed green (MX-222). Evidence copied to #19. |
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
1 participant
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.
Throwaway. A gate that has never failed is not known to be a gate — MX-222 exists because a green check proved less than it looked like it did. This branch breaks something on purpose so the new job is observed failing, and will be closed and deleted as soon as the red is read.