fix(docs): add Stop hook to lifecycle placement table

[brandisher]

Summary

• Adds missing Stop stage row to the lifecycle placement table in CONTRIBUTING.md
• The repo has an active Stop hook (scripts/lint-markdown.sh) for markdown linting, but it wasn't documented in the table
• Addresses post-merge feedback from @fonta-rh on docs: add architectural guidelines for hooks, test coverage, and review process #77

Test plan

• Verify CONTRIBUTING.md passes markdownlint
• Confirm the new row accurately reflects the registered Stop hook in .claude/settings.json

🤖 Generated with Claude Code

Summary by CodeRabbit

• Documentation
  • Updated contribution guidelines with session-end checkpoint specifications for code quality enforcement procedures.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: c571723b-c549-4286-b64d-5f7fd2afaaaa

📥 Commits

Reviewing files that changed from the base of the PR and between 094e6c8 and 6175382.

📒 Files selected for processing (1)

• CONTRIBUTING.md

---

Walkthrough

Documentation update to contribution guidance adding a session-end checkpoint specification. The change clarifies that markdown linting and cleanup tasks should be implemented via a repo-level Stop hook, extending the existing hook placement guidance.

Changes

Cohort / File(s)	Summary
Documentation CONTRIBUTING.md	Added session-end protection checkpoint details specifying repo-level Stop hook usage for markdown linting and cleanup tasks.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title 'fix(docs): add Stop hook to lifecycle placement table' directly matches the main change: adding a missing Stop stage row to the lifecycle table in CONTRIBUTING.md documentation.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[fonta-rh]

Looks good — accurately documents the existing Stop hook for markdown linting that was missing from the lifecycle table. Thanks for the quick turnaround on the feedback.

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: brandisher, fonta-rh

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [brandisher,fonta-rh]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment