You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
SkillBehaviors gets undefined behaviors prop on raw Skill snapshots #586
Since run 37763046034 (attempt 1, 2026-10-08T10:22Z, commit 6b120f6#573) through latest main run 37807825636 (attempt 1), job site / test, step Run pnpm test:run, test/unit/skill-detail-duplicate-canonical-path.nuxt.test.ts logs [Vue warn]: Invalid prop: type check failed for prop "behaviors". Expected Array | Null, got Undefined at <SkillBehaviors behaviors=undefined>, because SkillDetail.vue binds :behaviors="data.sourceFacts.behaviors" under v-if="data.raw || behaviors.length" and raw snapshots lack the field; the binding must coalesce undefined to null.
Verify with:pnpm exec vitest run test/unit/skill-detail-duplicate-canonical-path.nuxt.test.ts 2>&1 | grep -c 'Invalid prop: type check failed for prop "behaviors"' returns 0
Estimated to change 1 file.
The ci-review routine opened this issue automatically on its 2026-10-08T21:00:00.000Z run. It is not Harlan's own report. Close it to reject the proposal, and the reason you give stops it being offered again.
Summary: Verified in code: PR fix(registry): preserve root Skill snapshots #573 (commit 6b120f6) changed SkillDetail.vue:2338 from the coalescing computed behaviors to :behaviors="data.sourceFacts.behaviors", and widened the _SkillBehaviors.vue prop to SkillBehavior[] | null. The `v-if=
Next action: In an Issue-work agent: change layers/registry/app/components/SkillDetail.vue:2338 to :behaviors="data.sourceFacts.behaviors ?? null", then verify with pnpm exec vitest run test/unit/skill-detail-duplicate-canonical-path.nuxt.test.ts 2>&1 | grep -c 'Invalid prop: type check failed for prop \"behaviors\"' returning 0.
Related pull requests:fix(registry): preserve root Skill snapshots #573: Introduced the regression: swapped the coalescing behaviors computed for the direct data.sourceFacts.behaviors binding and widened the prop type to Array | Null without coalescing undefined.
Activity
harlan-github-agent commentedon Oct 8, 2026
🤖 ISSUE TRIAGE
behaviorsto:behaviors="data.sourceFacts.behaviors", and widened the _SkillBehaviors.vue prop toSkillBehavior[] | null. The `v-if=:behaviors="data.sourceFacts.behaviors ?? null", then verify withpnpm exec vitest run test/unit/skill-detail-duplicate-canonical-path.nuxt.test.ts 2>&1 | grep -c 'Invalid prop: type check failed for prop \"behaviors\"'returning 0.behaviorscomputed for the direct data.sourceFacts.behaviors binding and widened the prop type to Array | Null without coalescing undefined.