fix(fp-check): remove top-level Stop hook to stop firing on unrelated sessions (#143)

[remiforall]

Summary

Closes #143.

The Stop hook in plugins/fp-check/hooks/hooks.json is registered with "matcher": "*", so it fires on every Claude Code session stop — including sessions that have nothing to do with fp-check verification. This burns a full 30-second LLM turn on every unrelated stop and pollutes the parent context with stop-feedback blocks.

The hook's own fallback rule ("If the conversation is not about fp-check verification at all, return 'approve'") does not prevent the issue: the hook still fires, the model still runs, the turn is still consumed — just to discover that it should no-op.

Fix

Removes the top-level Stop hook entirely (option 3 of three workarounds discussed in #143).

The companion SubagentStop hook stays. It is the right place to enforce structured-output completeness, because it only matches actual fp-check subagents (data-flow-analyzer, exploitability-verifier, poc-builder) and never fires on unrelated sessions. Verification rigor is preserved at the level where it is actionable.

Impact

• Verification rigor: unchanged. SubagentStop continues to enforce phase-output completeness on every fp-check subagent.
• Unrelated sessions: no longer pay the cost of a full LLM turn at every Stop.
• Plugin version bumped from 1.0.0 to 1.0.1 (per repo CLAUDE.md guidance: substantive change to hook behavior).

Validation

$ python3 .github/scripts/validate_codex_skills.py
Validated 73 plugin skills against 74 Codex entries successfully.

CODEOWNERS unchanged (/plugins/fp-check/ @ahpaleus @dguido).

Test plan

• Diff reviewed for completeness (3 files: hooks.json + 2 version bumps)
• Codex skills validator passes
• No CODEOWNERS change required
• CI green
• Reviewer ack from @ahpaleus or @dguido that removing the top-level Stop hook is acceptable (per @GeeksikhSecurity's own recommendation in fp-check Stop hook with matcher "*" fires on every unrelated session, causing forced LLM turns #143: "Option 3 is the simplest and loses no real coverage.")

[claude]

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}