Skip to content

SkillBehaviors gets undefined behaviors prop on raw Skill snapshots #586

Closed

Description

@harlan-github-agent

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.

Target: layers/registry/app/components/SkillDetail.vue

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.

Activity

  1. harlan-github-agent commented on Oct 8, 2026

    @harlan-github-agent
    ContributorAuthor

    🤖 ISSUE TRIAGE

    Harlan Agent Kit posted this automated triage. It is not Harlan's personal assessment or commitment. AI open source policy.

    • Route: Ready to implement
    • Difficulty: 1/5
    • Impact: 2/5
    • Reproduction: Yes
    • Codebase review: Not needed
    • 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.
  2. added
    harlan-agent-ready-to-implementIssue triage found bounded work ready for an implementation Agent.
    harlan-agent-runningAn Agent holds a Task on this issue or pull request right now.
    and removed
    harlan-agent-runningAn Agent holds a Task on this issue or pull request right now.
    on Oct 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions