diff --git a/STATUS.md b/STATUS.md index 79121fc..618a2dd 100644 --- a/STATUS.md +++ b/STATUS.md @@ -1,6 +1,6 @@ # Automotive Skills Suite β€” STATUS -_Generated: 2026-05-20 by autonomous daily run (POLISH mode)._ +_Generated: 2026-05-21 by autonomous daily run (POLISH mode)._ **Builders:** 76 Β· **Reviewers:** 76 Β· **Paired:** 76/76 (100.0%) @@ -20,7 +20,7 @@ _Generated: 2026-05-20 by autonomous daily run (POLISH mode)._ | `autosar-bsw-config-builder.skill` | autosar | `autosar-bsw-config-checklist-reviewer.skill` | 2026-05-02 | 🟒 paired & fresh | | `autosar-composition-builder.skill` | autosar | `autosar-composition-checklist-reviewer.skill` | 2026-05-02 | 🟒 paired & fresh | | `autosar-rte-mapping-builder.skill` | autosar | `autosar-rte-mapping-checklist-reviewer.skill` | 2026-05-02 | 🟒 paired & fresh | -| `autosar-swc-builder.skill` | autosar | `autosar-swc-checklist-reviewer.skill` | 2026-05-02 | 🟒 paired & fresh | +| `autosar-swc-builder.skill` | autosar | `autosar-swc-checklist-reviewer.skill` | 2026-05-21 | 🟒 paired & fresh | | `bus-load-analysis-builder.skill` | comms | `bus-load-analysis-checklist-reviewer.skill` | 2026-05-02 | 🟒 paired & fresh | | `calibration-data-exchange-builder.skill` | calibration | `calibration-data-exchange-checklist-reviewer.skill` | 2026-05-02 | 🟒 paired & fresh | | `cdd-builder.skill` | diagnostics | `cdd-checklist-reviewer.skill` | 2026-05-02 | 🟒 paired & fresh | @@ -87,5 +87,5 @@ _Generated: 2026-05-20 by autonomous daily run (POLISH mode)._ - Total builders: **76** | Total reviewers: **76** | Paired ratio: **100.0%** - Flags: 🟒 76 fresh Β· 🟑 0 stale (>30d) Β· πŸ”΄ 0 unpaired -- Domain spread: safety=14, quality=8, comms=8, program-mgmt=6, cyber=6, autosar=5, diagnostics=5, v&v=5, aspice=4, sysml=4, calibration=3, mbse=3, sotif=3, other=2 +- Domain spread: safety=14, comms=8, quality=8, cyber=6, program-mgmt=6, autosar=5, diagnostics=5, v&v=5, aspice=4, sysml=4, calibration=3, mbse=3, sotif=3, other=2 - Classifier note: domain map preserved from W20 canonical STATUS; W21 target #10 (`scripts/classify_skill.py`) will freeze this as a checked-in deterministic script. diff --git a/docs/AUTONOMOUS_LOG.md b/docs/AUTONOMOUS_LOG.md index 6666e7d..b9579bb 100644 --- a/docs/AUTONOMOUS_LOG.md +++ b/docs/AUTONOMOUS_LOG.md @@ -186,3 +186,18 @@ Standout finding is non-DoD and more impactful than the trigger gaps: the SKILL. - W21 target #5 (classifier freeze, `scripts/classify_skill.py` + golden-file test) still unserviced β€” carries forward. Three+ consecutive runs have hit classifier/regen bugs this would prevent. - Issue #8 (autosar-swc) will not get a pass this week if Thu services #9 β€” it carries to W22 PLAN. Note in Saturday's RELEASE notes. - Issue #2 ("goodd", empty body) still open and needs-triage β€” un-actioned again. POLISH is the wrong venue; flag for the human at Saturday's RELEASE if still open. + +## 2026-05-21 (autonomous run, POLISH) + +**Mode:** POLISH (Thursday) +**Action:** Polished `autosar-swc-builder.skill` per W21 target #3 (issue #8, autosar cluster) β€” and, unlike the W21 8d/dbc passes, an actual fix was applied to the `.skill` archive: the frontmatter `description` and `## Usage` trigger line were rewritten, the archive re-zipped. Wrote `docs/skill-polish-log/autosar-swc-builder.md` (43 lines). Regenerated STATUS.md. +**Files touched:** skills/autosar-swc-builder.skill, STATUS.md, docs/skill-polish-log/autosar-swc-builder.md, docs/AUTONOMOUS_LOG.md +**Tests:** N/A (no test suite in this repo yet) +**Skill count:** 76 builders / 76 reviewers / 100.0% paired +**Open issues:** 9 (#2 needs-triage + #3-#6 W20 targets + #7-#10 W21 targets) β€” unchanged; POLISH does not open/close issues. +**Notes:** Deviation from yesterday's follow-up, recorded deliberately: the 2026-05-20 dbc log recommended Thu service #9 uds-services-builder and let #8 autosar-swc slip to W22. I serviced #8 instead. Rationale β€” #8 is target #3 in the WEEK-W21 file (ahead of #9 in plan order), and on inspection autosar-swc-builder admitted a clean, in-allowlist small fix where the last two POLISH passes (8d, dbc) could only log proposals. Net effect is the same as the plan (one target serviced Thu, one slips to W22) β€” except the slipped target is now #9 uds-services, not #8, and #8 actually got fixed rather than just analysed. The fix: the description previously ended with the boilerplate self-trigger "Use this skill when the user mentions autosar swc builder" and never disambiguated Classic vs Adaptive. Rewrote it (723/1024 chars) to scope explicitly to Classic Platform, redirect Adaptive-platform traffic to `autosar-adaptive-app-builder`, and expand triggers to name sender-receiver/client-server port types, RTE events, ARXML SWC output, and the casual phrasing "define an AUTOSAR component" β€” satisfying all three issue-#8 DoD bullets. Body prose left untouched (no refactor); one residual non-redirecting "AUTOSAR Classic SWC" line in the body was left as cosmetic. Per the dbc follow-up I diffed the SKILL.md against actual ZIP contents β€” autosar-swc-builder's SKILL.md has no "Files in this skill" tree, so the dbc-style examples/-drift miss does not apply here; archive ships 7 files under references/ and scripts/. +**Follow-ups:** +- W21 target #9 uds-services-builder now carries to W22 PLAN (was #8 per yesterday's plan; swapped as explained above). W22 PLAN should also still pull calibration/mbse/sysml per the WEEK-W21 spread note. +- W21 target #5/#10 (classifier freeze, `scripts/classify_skill.py` + golden-file test) STILL unserviced after the full Tue/Wed/Thu POLISH cycle β€” four+ consecutive runs have now flagged classifier/regen drift this would prevent. Strongly recommend W22 PLAN promote it to the Monday slot or have Saturday's RELEASE call it out. +- Consider a symmetry pass on `autosar-adaptive-app-builder` so its description cross-references autosar-swc-builder the same way (Classic↔Adaptive redirect now only points one direction). +- Issue #2 ("goodd", empty body) still open and needs-triage β€” un-actioned by POLISH again. Flag for the human at Saturday's RELEASE if still open. diff --git a/docs/skill-polish-log/autosar-swc-builder.md b/docs/skill-polish-log/autosar-swc-builder.md new file mode 100644 index 0000000..d7d4280 --- /dev/null +++ b/docs/skill-polish-log/autosar-swc-builder.md @@ -0,0 +1,43 @@ +# Polish log β€” autosar-swc-builder.skill + +## 2026-05-21 (autonomous POLISH run, W21 target #3 / issue #8) + +**Domain:** autosar Β· **Severity:** med Β· **Outcome:** small fix applied + +### What's good +- SKILL.md is well structured: clear Inputs (JSON schema), a 13-tab Output + contract, and a `/autosar-swc-builder` command stub. +- Correctly anchored to a concrete standard revision (AUTOSAR R22-11) and scoped + to a single application / sensor-actuator component β€” a sensible unit of work. +- Output tabs already enumerate sender-receiver, client-server and mode-switch + interfaces separately, so the workbook structure matched the DoD before edits. + +### What to fix +1. **Weak trigger sentence (med).** The frontmatter `description` ended with the + boilerplate "Use this skill when the user mentions autosar swc builder." β€” + the same low-value self-referential trigger seen across other builders. It + gave the classifier nothing to match beyond the literal skill name. +2. **Classic vs Adaptive not disambiguated (med).** The description said + "AUTOSAR Classic SWC" but never told the model (or the reader) that the + Adaptive Platform equivalent is a different skill. Risk: this skill fires on + Adaptive-platform requests it cannot serve. +3. **Port interface types not surfaced in triggers (low).** sender-receiver vs + client-server appeared only in the Inputs section, not in the trigger + surface β€” so a user asking by interface type would not reliably route here. + +### Edits applied this run +- Rewrote the `description` (now 723 chars, well under the 1024 limit): + explicitly says "Classic Platform ... only", points Adaptive traffic to + `autosar-adaptive-app-builder`, and expands triggers to name port-interface + types, RTE events, ARXML SWC output, and the casual phrasing "define an + AUTOSAR component". +- Updated the `## Usage` trigger line to match, with the same Classic-vs-Adaptive + redirect. +- Body prose (Inputs / Output / Commands) left untouched β€” no refactor. + +### Deferred (not done this run) +- The body still has one residual "AUTOSAR Classic SWC" line that does not carry + the Adaptive redirect; cosmetic only, low value, left for a future pass. +- Worth checking the Adaptive counterpart (`autosar-adaptive-app-builder`) so the + two descriptions cross-reference each other symmetrically β€” captured as a W22 + follow-up candidate. diff --git a/skills/autosar-swc-builder.skill b/skills/autosar-swc-builder.skill index ce321c9..5a88095 100644 Binary files a/skills/autosar-swc-builder.skill and b/skills/autosar-swc-builder.skill differ