Skip to content

P3: setup verify flags _find-sdlc-root.sh as non-executable — installer and verifier disagree about intent (both lanes) #703

Description

@BaseInfinity

Finding

Side finding from the #697 follow-up verification (2026-08-26, both lanes exercised live).

The installer deliberately writes .claude/hooks/_find-sdlc-root.sh without the executable bit — cli/init.js:34 omits executable: true (every other hook has it), correct since the file is only ever sourced. But the setup skill's verify-only check flags it as an actionable defect and prescribes chmod +x .claude/hooks/_find-sdlc-root.sh; on the latest lane the skill went ahead and applied that fix. Observed on both lanes: v1.87.0 (stable) and v1.99.2 (latest).

Why it matters (P3)

Harmless either way — sourcing ignores mode bits. But the wizard's own verification contradicts the wizard's own installer, so every fresh consumer who runs verify sees a phantom defect, and the skill "fixes" a mode the installer chose. One of the two is wrong about intent.

Ask

Pick one: teach the setup skill's verification that sourced helpers are exempt from the +x expectation, or mark the file executable: true in the manifest so installer and verifier agree. Either ends the contradiction.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions