Repository navigation
fix(ci): restaurar gates tras merge prematuro de #88 - #89
Merged
Merged
Conversation
Los schemas/fixtures ya no nombran herramientas de host prohibidas, y la suite application_contract queda enlazada a FR/NFR en alcance.
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: 1fe4b0fb-9ea1-49ff-92fc-1007c52d2d24) |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: 19a9d7c3-9876-44e5-9fea-edd3cd1a8dc7) |
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.
Summary
pacstrap/cmd/…); usan marcadores opacos (host-bootstrap,legacy-command-field).application_contractenlaza a FR-VALIDATE/MANIFEST/EXPORT + NFR-DET/OFF; 0 pruebas huérfanas.Por qué
gh pr merge --autofusionó #88 antes de que terminaran los jobs requeridos (la protección de rama no los bloqueó). Esta PR reparamain. A partir de aquí no se fusiona nada con checks en rojo.Test plan
contracts/**sin hits SHELLpython3 tools/traceability_matrix.py→ TODAS LAS METRICAS CUMPLEN, 0 huérfanascargo test -p archmaker-core --test application_contract --locked(8 passed)Note
Low Risk
Changes are schema wording, fixtures, inventory/traceability metadata, and agent policy—no runtime behavior paths. Main risk is contract enum drift if producers still emit old literal tokens.
Overview
Repairs CI gates broken after a premature merge by aligning ADR-0011 contract artifacts with the repo’s no-literal-shell policy and restoring traceability.
Contract schemas and fixtures no longer embed prohibited host-tool names (
pacstrap,cmd,bash, etc.). Descriptions and enums now use opaque process markers (host-bootstrap,legacy-command-field,host-image-build, …), with matching updates in invalid JSON examples andapplication_contracttest expectations.Traceability registers the eight
application_contractRust tests as typecontract, maps them to FR-VALIDATE/MANIFEST/EXPORT and NFR-DET/OFF viagen_test_inventory.py, and refreshestests-inventory.json/traceability-matrix.*(includingVAL-HTML_CATALOG_MATRIX).Governance:
AGENTS.mdadds an explicit rule—do not merge tomainwhile any CI check is failing or still pending.Reviewed by Cursor Bugbot for commit 4e5ed43. Configure here.