Skip to content

Commit 48c2f6a

Browse files
NubsCarsonclaude
andcommitted
fix(chip): scope yamllint indentation override + pin remaining objectives
Follow-up hygiene wins on top of 76b9dbe. packages/chip/.yamllint.yaml - Replace wholesale 'indentation: disable' with a per-path override. - Indentation is now enforced for hand-authored YAML in the chip tree. - The override list covers exactly the 11 generator-owned files whose PyYAML safe_dump output uses sequence indentation yamllint flags: board/kicad/e1-phone/production/ subtree, three dated audit YAMLs under board/kicad/e1-phone/, and docs/evidence/pd/e1-soc-pd-input-contract.yaml. - Verified by injecting a deliberate mis-indent into a hand-authored file: yamllint catches it; reverting restores a clean exit. packages/chip/scripts/check_e1_phone_board_package.py - check_end_to_end_readiness: pin the two remaining objectives to their authoritative artifacts. - component_height_and_enclosure_step → enclosure-fit-execution-package.yaml - supplier_footprints_pinouts_and_3d_models → supplier-to-kicad-evidence-map.yaml - Both anchors are real v1-schema, fail-closed artifacts with forbidden_claims. - All 9 end-to-end objectives are now pinned (was 7 of 9). Verified: docker run ... make e1-phone-board-package-check (green) docker run ... make clean ci-fast (ci-fast complete) yamllint . (exit 0) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 76b9dbe commit 48c2f6a

2 files changed

Lines changed: 14 additions & 4 deletions

File tree

packages/chip/.yamllint.yaml

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,15 @@ rules:
1616
allowed-values: ["true", "false", "on", "off"]
1717
comments:
1818
min-spaces-from-content: 1
19-
# The chip tree carries YAML from KiCad/package manifests, PyYAML-generated
20-
# evidence reports, and hand-authored docs. Their sequence indentation styles
21-
# intentionally differ; content validators cover schema/semantic correctness.
22-
indentation: disable
19+
# Indentation stays enforced for hand-authored YAML. The chip tree also
20+
# carries PyYAML-generated evidence reports whose default `safe_dump` block
21+
# style does not match yamllint's expected sequence indentation; relax the
22+
# rule only for those generator-owned subtrees and dated audit files. Schema
23+
# and content validators still enforce semantics.
24+
indentation:
25+
ignore: |
26+
board/kicad/e1-phone/production/
27+
board/kicad/e1-phone/e1-phone-objective-completion-audit-*.yaml
28+
board/kicad/e1-phone/e1-phone-readiness-unblock-register-*.yaml
29+
board/kicad/e1-phone/kicad-route-readiness-inventory-*.yaml
30+
docs/evidence/pd/e1-soc-pd-input-contract.yaml

packages/chip/scripts/check_e1_phone_board_package.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13870,7 +13870,9 @@ def check_end_to_end_readiness() -> None:
1387013870
"usb_c_power_volume_hardware": "board/kicad/e1-phone/usb-sidekey-selection-wiring-decision.yaml",
1387113871
"off_the_shelf_wireless_modules": "board/kicad/e1-phone/wireless-module-release-execution.yaml",
1387213872
"board_size_power_rf_thermal_optimization": "board/kicad/e1-phone/layout-optimization-execution.yaml",
13873+
"supplier_footprints_pinouts_and_3d_models": "board/kicad/e1-phone/supplier-to-kicad-evidence-map.yaml",
1387313874
"schematic_and_pcb_routed_release": "board/kicad/e1-phone/routed-pcb-implementation-execution.yaml",
13875+
"component_height_and_enclosure_step": "board/kicad/e1-phone/enclosure-fit-execution-package.yaml",
1387413876
"manufacturing_and_factory_release": "board/kicad/e1-phone/production-factory-release-execution.yaml",
1387513877
}
1387613878
for objective, evidence_path in expected_evidence.items():

0 commit comments

Comments
 (0)