From 985078096bbd1ea8870344928a88ad03ed7ada6d Mon Sep 17 00:00:00 2001 From: Automotive Skills Autonomous Date: Tue, 26 May 2026 11:08:18 +0000 Subject: [PATCH] auto(polish): W22 #1 uds-services-builder pass, regen STATUS Autonomous daily run. - POLISH: rewrote uds-services-builder frontmatter description (285->953 chars) with broad triggers + sibling-skill redirects to odx/cdd/dtc-catalog/dem-config - POLISH: aligned body trigger sentence with new description - LOG: docs/skill-polish-log/uds-services-builder.md created - STATUS regen: 76 builders / 76 reviewers / 100% paired (pairing overrides preserved) Generated by automotive-skills-daily-standup scheduled task. --- STATUS.md | 8 +-- docs/AUTONOMOUS_LOG.md | 15 +++++ docs/skill-polish-log/uds-services-builder.md | 60 ++++++++++++++++++ skills/uds-services-builder.skill | Bin 9979 -> 10255 bytes 4 files changed, 79 insertions(+), 4 deletions(-) create mode 100644 docs/skill-polish-log/uds-services-builder.md 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 160bbd1b4563bec7b2245697456309c3d7307e5a..30a9c25c766084d0dae55146c7462672471b64db 100644 GIT binary patch delta 1519 zcmV@6aWAK2mqUYqFl_xDZFR}007<%00146|0EcHQ%gxqOfGF? z)tAk3+cpr#?|h1#$-z#Bq%1p46Zhnil8!sA?UC)aHyR=!2{B1<08kQh>O=Gi`y|~3 z_-QMxC(}z$9uwHb?&7!qg`y}53YLp=sJMxY&|8@b6D=#5aiIe)OsZvRr7F(hM^OmP ztN>O3tGKk07L3<$Kfi^4#u8FyS*F&oQ~GhK)FYHa!{zNYOy8cKeu$=!s;$tnSc9p` zQfVtVaLLw%GFGOo8w1BD`{_w^vVZ^Xb%CE!8X@@PQPi-( z|1vZSp%80ktU~CZNKqptHiIPt^|liiX4DhKEY&>($gL z92V_q4Z=~VO+7<@XlvGg!K4r6j7ecGqZmq&%9X_P=w|-<<4?e(@QRqJSO(=rGlP~` zeEBHgh<3Y#dk@By$`uA(ABAwV+b~NR8Q=;ppj~QlI&xtg?Jz4WT)VS3%ta2mLNIbcXkiY{T;vgU!Af@4aS&mj2d@q{ zdGOyhHbZ{@vAGX5v~gs*HugvY8}?=2-dvPKW06X7L`CuVLCKI@VTCr&+bamZwZCBU z!Un@5U))-Mxc4hf^^z8nMVjFyp`UN=qZmH@FQ^S(y@Km4)miL<;6^K6(QsfeJ&P<` z!ag(mo=7^!m#c0$B!Ul(m!8jMP1HLLp0UzanpPX;`~1e^s6%lj3)wR~c{-5BB>Utl z?QZg`BPD#!^S?|JU#82VtZWecbbEaTE13z{NxM0J4N z{H5*n;kV5$!4z#$RQVD`2>mv7O$K#E_stj|bQ7C;yTHKNrx5VLwzEF5jIE?i% zLaXRu{ig%y;41>`8I5`$6&ca|R3WN>APhm~SSVYbk5 zVB{AB4(AVcz%o8;~o1ga(J1^~j zcUzajKQjkxnBY6I^r~~&sg64xsqMt)0KdVz*v+3=GoO8ii0f|-*E!*O+2J}KCnoql zc2jF)+)+6cxZr)?T}P7zvp_q4MC2E*n>Pd9{Mo<4$^KKu%%;^=rPp}+7ne^k zMSgx|ndCmbMU^#SI;AD{+a(GW$;aDdvY=BDs72R&-sZpyQlftCTjOo>{BYdg8@9#{ zczAeNVst@SqZk+9e~FpNx_PZ7_?U~!8vS5A@n?fulxvun#v)kDgwS1k|iILBPKHfp&pVYCX>${3X_N? VDgyW=k|i;d@6aWAK2mqUYqFh%3h&WUP001Nk00146z6BV6Q%gxqOfGF? z-ImL8+cp%2_jrmklTFeQY0FO1#Jh}?cGPKYk6gD)8X_SIF$Hh{P!hB1L-YyzB>e&K zrm3Ew3u{ae=ipqv^Mgr}Fva)cj+(+IR+xj#g-te%tO{XRA#84B?W9(B^s`XHa3_c> z;!Po4VgxT*dR{(JXGuAKcf8Ui9kqGgX#GmHFtmDlp!xfoo6pIda(xg+s*-qBk!|W) z8z%}XBrlb=PUgKUJHzWuvTaoEkH_P3wZ!i_{92JWM!J@Gj;)p|aS&Blw-&pp5VcT+ zP`UJuxGKoWy+5|*2I)FsEN!(RPL7*WIQqJHPU+bSayx0szDk6DciahuXF&~OqPKxkCeWp2=c{7!J~gunMq$j%|(GX>8;*wxR(u1h=ob zYQ00`1tZUdYvCB#qG46Wi^7Dq*1Oi8uk<`m*LKGYDY{HG`wa#Se!IG6le)q^o_JHa z*);PRiIelF7eh1f7E|XyesmQ&m<=6-qx0DtQ1Ep+A3E=#kVV$D5~hCSua0P@@b%#o zv)@WTP|?1B?{yf-5fP;7j0|qQS99(a&67$HW;=QjC%ADTo1ytJo9I1@B*FCjya1ah zu3iOTt4fVGyFJd{3Ij2S73p!ydfc*r8+_fb7gR~>Vhnr16)GHTB8k};40b=>%<%UE zhWVBbylPNEsb~-3kO)evB3MRPvq{Uc2&<1B;^Sz4;K0i701g_93D{IRe+?^f^cxqT z?xy|gDHqi+?ssgS((lp%^?&~Qo0d9nyiOqj$9A8I8WeZY-9a%!;QD94b?I?k4Y-~QtYtr@L7~r#2P&rm$3CTld{fkK zU(`|1x+KrA3~3HPTJ|}m`(E|dz3MGIhQZo{*y9&(FC)v+&%pfJw>oIv<9L;Mn+cBN z)PY-pEMNyjev7*KIMK~t1B$`^r;6KMudUjDlsNr+$cN3r&mX*!MNDtqR9%=sX+8V> zD<3MzM>n}t{!{?99)>U4Og;w2+ocev-uTd6v4Q(%%S%ka%gf6K(XqOP7%Ph3bM~t2 zzDP2Bu%vYhKZqy&X0b(wfl2+f=&jkmP)h>@6aWAK2mnZXqFmpr)i==)0084i001nr zz6F^e3ReP%I8+1x03-^NVJ2Gwj|7vy1um29CMp7|6_dXOACosHGXjkrlfMNYliVH= Zlbk0i0^}r