Repository navigation
cloud: fix typecheck in the devbox size scripts - #11678
Merged
Merged
Conversation
FirewallSpec needs the literal destination.public: true, and the promoted size set is keyed by the size-name-or-empty string sizeKey returns. Follow-up to #11664. Claude-Session: https://claude.ai/code/session_017SYRh8isujtDXJPoCg2GU5
|
Warning Review limit reachedNext included review available in 9 minutes. View limit detailsLimit details: You’ve used all 10 included reviews currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Team Run ID: 📒 Files selected for processing (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
|
All contributors have signed the CLA ✍️ ✅ |
rustybret
pushed a commit
to rustybret/bmux
that referenced
this pull request
Sep 2, 2026
5db1af3 Merge pull request manaflow-ai#11589 from manaflow-ai/feat-replay-tombstone c98bf28 Check manifest size problems without the arrayContaining shim (manaflow-ai#11681) dab9d7f fix(remote): retain all lanes on remote reset 873f9a3 test(remote): cover reset tombstone data lanes 9598aad fix(remote): retain handshake lane on pending open teardown 72ab166 test(remote): cover pending open handshake tombstone ecf12d7 fix(remote): retain legal lanes for rejected opens 156fb23 test(remote): cover open-limit data lane tombstone 096119e test(remote): assert lane-specific tombstone retention 0e0bb62 fix(remote): scope tombstones to the closed lane e152bb4 test(remote): cover cross-lane close tombstone retention 62f8e4d test(remote): churn tunnel tombstones to configured bound edd0b97 fix(remote): retain tunnel tombstones through replay window 84640ad test(remote): expose tunnel tombstone churn 1ba8941 fix(remote): retain legal lanes on removal 28599ae test(remote): retain removal tombstones across legal lanes 09bffdc fix(remote): retain all legal lanes on drop bf73e72 test(remote): retain dropped stream legal lane tombstone e09b07c fix(remote): retain dropped stream lane tombstones 3c282a2 test(remote): retain dropped stream tombstone lane 0f6e222 fix(remote): match tombstones by lane 0eea0c1 test(remote): reject wrong-lane tombstone frames 051e9ca fix(remote): remove obsolete tombstone helper 5bddc99 fix(remote): scope tombstones to affected lanes eac37bc fix(remote): retain tombstones per replay lane 8480e7b test(remote): expose replay tombstone churn e341deb Fix main CI: guard exemption, manifest typecheck, docs-channel Vercel config, device registry test isolation (manaflow-ai#11648) 3425245 web: bake the cmux-tui daemon into the Freestyle devbox; create is vms.create plus one file write (manaflow-ai#11666) e941f22 Add Copy to notification context menus (manaflow-ai#11677) 792b9cb cmux-tui: fix clippy 1.95 lints so the full gate is green again (manaflow-ai#11625) 8711a34 cloud: fix typecheck in the devbox size scripts (manaflow-ai#11678)
This branch was successfully deployed
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.
Follow-up to #11664:
tsc --noEmitfailed on the new size scripts.FirewallSpecneeds the literaldestination.public: true, and the promoted-size set is keyed by the stringsizeKeyreturns.https://claude.ai/code/session_017SYRh8isujtDXJPoCg2GU5
Summary by cubic
Fixes a typecheck failure in the devbox size scripts that followed the size promotion work.
FirewallSpectype sodestination.public: trueis typed correctly.Set<string>to match the string returned bysizeKey.Written for commit fcd5a01. Summary will update on new commits.