Repository navigation
fix(bugbot): validate Compose width semantics and suggested constraints - #744
Conversation
…ted-modifier-order-removes-width-cap: ground Compose width findings and replacement constraints
|
To preview the documentation for this pull request, visit the following URL:
|
Bugbot: review complete
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #744 +/- ##
========================================
Coverage 94.50% 94.50%
========================================
Files 1035 1035
Lines 33298 33298
Branches 9824 9824
========================================
Hits 31467 31467
Misses 699 699
Partials 1132 1132
🚀 New features to boost your workflow:
|
Copilot approval · Human review neededStatus: A protected workflow, policy, or trust-boundary file changed. Technical evidenceRevision: |
|
/copilot sync-branch --from develop |
Request could not be completed
Action: Inspect the current state and retry the failed step. Retained state: Existing persisted state and completed external effects were preserved. Error code: Retryable: Yes Reference: |
…ted-modifier-order-removes-width-cap: Merge develop into #720 preserving both Compose review contracts Resolve the eleven conflicts between #720 width/padding review grounding and #722 framework-owned startup cleanup, retaining both sets of fixtures and reference predictions (48 cases), both specification sections, their evidence and non-conflicting parent changes. Regenerate the catalog and all bundles from the combined source. Recovery follows the maintainer request to resolve the conflicts after Action run 40482345521 failed verification because the extension dependencies were unavailable in that runner environment. The Action did not push a merge. Child: c9df3cf Parent develop: 8de9b63 Plan: preserve both histories, run the existing verification gates and hooks, and publish a normal push to the existing managed ref without bypass or force. Validation: 725 suites and 12,511 tests passed (35 configured skips), all coverage budgets and the unchanged setup ledger passed; the modified prompt module has 100% coverage. Extension: 30 files / 267 tests passed with 100% coverage in every metric. Focused combined review regression selection: 34 passed. Build, typecheck, lint, specifications/catalog and documentation passed.
Bugbot now grounds Compose width findings and suggested fixes in actual constraints, modifier order, padding, and interactive bounds. The change addresses issue #720’s false overflow claim while preserving detection of real overflow and lost width caps.
What changed
Validation
Review notes
Closes #720