diff --git a/STATUS.md b/STATUS.md index 44e984e..0cf4e32 100644 --- a/STATUS.md +++ b/STATUS.md @@ -1,6 +1,6 @@ # STATUS β€” automotive-skills-suite -_Auto-generated 2026-05-25 by automotive-skills-daily-standup (PLAN run)._ +_Auto-generated 2026-05-26 by automotive-skills-daily-standup (POLISH run)._ | Builder | Domain | Paired Reviewer | Last Touched | Flag | |---|---|---|---|---| @@ -55,9 +55,9 @@ _Auto-generated 2026-05-25 by automotive-skills-daily-standup (PLAN run)._ | pfmea-builder.skill | quality | pfmea-checklist-reviewer.skill | 2026-05-01 | 🟒 paired & touched ≀30 days | | ppap-package-builder.skill | quality | ppap-checklist-reviewer.skill | 2026-05-01 | 🟒 paired & touched ≀30 days | | safety-case-builder.skill | safety | safety-case-checklist-reviewer.skill | 2026-05-01 | 🟒 paired & touched ≀30 days | -| safety-gate-review-builder.skill | program-mgmt | safety-gate-review-checklist-reviewer.skill | 2026-05-02 | 🟒 paired & touched ≀30 days | +| safety-gate-review-builder.skill | safety | safety-gate-review-checklist-reviewer.skill | 2026-05-02 | 🟒 paired & touched ≀30 days | | safety-plan-builder.skill | safety | safety-plan-checklist-reviewer.skill | 2026-05-01 | 🟒 paired & touched ≀30 days | -| safety-program-risk-register-builder.skill | program-mgmt | safety-program-risk-register-checklist-reviewer.skill | 2026-05-02 | 🟒 paired & touched ≀30 days | +| safety-program-risk-register-builder.skill | safety | safety-program-risk-register-checklist-reviewer.skill | 2026-05-02 | 🟒 paired & touched ≀30 days | | secure-coding-guidelines-builder.skill | cyber | secure-coding-guidelines-checklist-reviewer.skill | 2026-05-01 | 🟒 paired & touched ≀30 days | | sotif-analysis-builder.skill | sotif | sotif-analysis-checklist-reviewer.skill | 2026-05-01 | 🟒 paired & touched ≀30 days | | sotif-validation-strategy-builder.skill | sotif | sotif-validation-strategy-checklist-reviewer.skill | 2026-05-01 | 🟒 paired & touched ≀30 days | @@ -87,4 +87,4 @@ _Auto-generated 2026-05-25 by automotive-skills-daily-standup (PLAN run)._ - Total reviewers: 76 - Paired: 76 / 76 (100%) - 🟒 76 paired & fresh Β· 🟑 0 stale (30+ days) Β· πŸ”΄ 0 orphan builders -- Generated 2026-05-25 β€” next refresh on the following autonomous run. +- Generated 2026-05-26 β€” next refresh on the following autonomous run. diff --git a/docs/AUTONOMOUS_LOG.md b/docs/AUTONOMOUS_LOG.md index 8bc8341..d6f470a 100644 --- a/docs/AUTONOMOUS_LOG.md +++ b/docs/AUTONOMOUS_LOG.md @@ -263,3 +263,18 @@ Standout finding is non-DoD and more impactful than the trigger gaps: the SKILL. - Consider closing stale polish issues #6/#7/#8 β€” their commits landed in W21 but issues stayed open. - Triage or close issue #2 ("goodd") β€” clearly junk. - Decide whether tooling issues #10/#11 belong in a weekly plan or a separate backlog. + +## 2026-05-26 (autonomous run, POLISH) + +**Mode:** POLISH +**Action:** Executed W22 polish target #1 (uds-services-builder) β€” frontmatter description rewritten from 285β†’953 chars with broader triggers + sibling-skill redirects; body trigger sentence aligned. +**Files touched:** skills/uds-services-builder.skill (inner SKILL.md only), docs/skill-polish-log/uds-services-builder.md, STATUS.md, docs/AUTONOMOUS_LOG.md +**Tests:** N/A (no test suite in this repo yet) +**Skill count:** 76 builders / 76 reviewers / 100% paired +**Open issues:** 11 +**Notes:** Picked uds-services as today's target per W22 priority order β€” it has been the #1 carryover for two weeks (issue #9 open since W21, never serviced) and was the only W22 polish target with no existing polish-log entry. The frontmatter description previously ended on the boilerplate trigger "Use this skill when the user mentions uds services builder" β€” i.e. the classifier had nothing to anchor on beyond the literal skill name. New description names DIDs, RIDs, SecurityAccess, NRC, P2/P2*/S3 timing, the three session types, the 0x10-0x86 service range, three casual phrasings, and explicit sibling redirects to odx-builder / cdd-builder / dtc-catalog-builder / dem-config-builder so the classifier can route correctly across the diagnostics neighbourhood. Rebuilt the .skill zip preserving the four sibling scripts byte-for-byte (only SKILL.md changed). STATUS regen: hit the prior-run override edge case again where item-definitionβ†’item-def and ppap-packageβ†’ppap need explicit pairing overrides; preserved both to keep the 76/76 paired headline intact. +**Follow-ups:** +- Tomorrow (Wed): W22 target #2 hara-builder.skill (issue #3, three-week carryover). +- Issue #9 (uds-services polish target) is now actually done β€” flag for human to close, or let Sunday TRIAGE add a "completed-this-week" comment. +- The STATUS classifier still embeds pairing overrides inline (#10 still unserviced after 7+ runs); recommend Monday W23 PLAN promote #10 to a tooling slot rather than another polish target. +- Worth a symmetric pass on uds-services-checklist-reviewer.skill so the reviewer description carries the same vocabulary β€” captured as a W23 follow-up. diff --git a/docs/skill-polish-log/uds-services-builder.md b/docs/skill-polish-log/uds-services-builder.md new file mode 100644 index 0000000..6cc5af0 --- /dev/null +++ b/docs/skill-polish-log/uds-services-builder.md @@ -0,0 +1,60 @@ +# Polish log β€” uds-services-builder.skill + +## 2026-05-26 (autonomous POLISH run, W22 target #1 / issue #9, two-week carryover) + +**Domain:** diagnostics Β· **Severity:** med Β· **Outcome:** small fix applied + +### What's good +- SKILL.md cleanly structured: an Input JSON schema, a 13-tab Output contract + enumerating service inventory / DID / RID / SecurityAccess / NRC / timing / + memory layout, and a `python scripts/generate_uds.py input.json output.xlsx` + Usage snippet. The 13-tab output mirrors what diagnostics teams actually + hand to the OEM, so the workbook structure was already DoD-aligned before + edits. +- Correctly scoped to ISO 14229-1 (protocol-neutral UDS), keeping the skill + off ISO 14229-3 (CAN-TP) and ISO 14229-5 (DoIP) which would belong in + separate skills. +- Inner zip is healthy: SKILL.md + `scripts/generate_uds.py` + `recalc.py` + + a small `scripts/office/` helper package. No dead files. + +### What to fix +1. **Trigger sentence was nearly empty (med).** The frontmatter `description` + ended with the boilerplate "Use this skill when the user mentions uds + services builder." β€” i.e. the classifier had nothing to match on except + the literal skill name. UDS-relevant requests phrased with the actual + vocabulary the field uses (DID catalog, SecurityAccess, NRC, P2/P2*/S3, + programming session) were not going to route here reliably. +2. **No disambiguation against neighbour skills (med).** The repo also has + `odx-builder`, `cdd-builder`, `dtc-catalog-builder` and `dem-config-builder` + β€” all in the same broad neighbourhood. With a weak description the + classifier could mis-route between them. Worth being explicit about who + owns what. +3. **Service range and session names hidden in the body (low).** The 0x10-0x86 + range and the default / programming / extended session vocabulary appeared + only in the body β€” surfacing them in the description gives the classifier + harder anchors. + +### Edits applied this run +- Rewrote the frontmatter `description` (now 953 chars, well under the 1024 + limit): expanded triggers to name DIDs, RIDs, SecurityAccess, NRC, the + three session types, P2/P2*/S3 timing, the 0x10-0x86 service range, and + three casual phrasings ('spec the diagnostics for this ECU', 'what DIDs + does this ECU support', 'write up the UDS services'). Added an explicit + sibling-skill redirect for ODX (odx-builder), CANdela (cdd-builder), DTC + inventory (dtc-catalog-builder), and the AUTOSAR Dem runtime layer + (dem-config-builder). +- Rewrote the `## Overview`-area body trigger sentence to match: same vocab, + same sibling redirects, slightly shorter to avoid prose bloat. +- Body Inputs / Output / Usage sections left untouched β€” no refactor. + +### Deferred (not done this run) +- The Output section currently flags `12. Validation Rules` as "Input + validation and constraints" but doesn't say which constraints get checked β€” + a one-line example would make the contract clearer. Cosmetic, low value, + left for a future pass. +- Worth a symmetric pass on `uds-services-checklist-reviewer.skill` so the + reviewer's description carries the same vocabulary; captured as a W23 + follow-up candidate. +- The Adaptive UDS / DoIP angle (ISO 14229-5) is not in scope for this skill + and not in scope for this repo today β€” flag for product judgement, not for + a polish pass. diff --git a/skills/uds-services-builder.skill b/skills/uds-services-builder.skill index 160bbd1..30a9c25 100644 Binary files a/skills/uds-services-builder.skill and b/skills/uds-services-builder.skill differ