Hygiene after v0.6.7: cppqualcheck reports timeouts as timeouts, #382 review notes, CONTRIBUTING map rows

[joyful-ii-V-I]

Small hygiene follow-up after v0.6.7 and #382. Docs, tests and comments only; every answer is byte-identical (31/31 argv vs main, --quality-delta gates 0).

1. test/cppqualcheck.sh: a timed-out call no longer reads as a wrong count. The repo-root arms ran each call cold under a 30 s alarm with stderr discarded, so on a slow runner (the ASan macOS leg on #382 took 28–30 s per arm) a killed call printed nothing and the arm said expected 32. Now:

• the six repo-root calls share one temporary cache per verb (one cold parse, then warm calls; --uses and --callers read different cache tiers, so one file per verb);
• a timeout prints FAIL TIMEOUT (rc=142) and any other non-zero exit FAIL ERROR, naming the call;
• sanitizer builds get a 90 s alarm (the gate's existing nm detection).
  No count or assertion changed. With the alarm forced to 1 s, all six arms print TIMEOUT; before, they printed expected N.

2. #382 review notes (thanks again @LaimaWu):

• stale references to the deleted capturePythonImportBinds in comments (src/ingest_importcap.h, src/graph.h);
• test/pyimportprecisecheck.sh: the "vocabulary compiled in" arm now greps Python-only patterns, so it fails on a binary without them (checked on v0.6.5 and v0.6.7);
• the refactor(#358): move Python import edges to shared captures #382 CHANGELOG entry now says the order of a file's bind records changed (no reader depends on it), plus a stray blank line;
• queries/python/tags.scm: "the C++ side (ingest_importcap.h) imposes" instead of "C++ imposes".

3. CONTRIBUTING family map: row 1 lists Python on @import.path (#382 merged), and row 2 names Python's current bind producer.

Gates: cppqualcheck (plain and ASan: 97 PASS each), pyimportprecisecheck, qschemetripcheck, readmedriftcheck, docanchorcheck, ripwirepubliccheck, gateexitcheck, manifestcheck.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Documentation
  • Clarified that Python import processing captures imported names and aliases, and that persisted content and extraction versions remain unchanged while bind-record ordering differs.
  • Documented that caches from the previous release continue to produce identical answers.
• Tests
  • Strengthened checks for Python import capture patterns and previous-release cache compatibility.
  • Improved test-gate reporting for timeouts and other failed calls, including sanitizer runs.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository: redhat-et/ripwire/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: a7ad98fd-ef5e-4591-8223-9643ddb8cc31

📥 Commits

Reviewing files that changed from the base of the PR and between 035b0bc and 00c30a6.

📒 Files selected for processing (7)

• CHANGELOG.md
• CONTRIBUTING.md
• queries/python/tags.scm
• src/graph.h
• src/ingest_importcap.h
• test/cppqualcheck.sh
• test/pyimportprecisecheck.sh

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 7 remain after this review.

📝 Walkthrough

The changes update Python import capture descriptions and checks. They also adjust cppqualcheck.sh timeouts, command failure handling, and cached repo-root checks.

Changes: Python import capture

Layer / File(s)	Summary
Capture descriptions and check queries/python/tags.scm, src/graph.h, src/ingest_importcap.h, CONTRIBUTING.md, test/pyimportprecisecheck.sh, CHANGELOG.md	Comments and documentation describe Python import captures and bind emission. The binary check requires both exact Python capture patterns. The changelog specifies unchanged record content and extraction version, changed bind-record order, and previous-release cache acceptance.

cppqualcheck gate

Layer / File(s)	Summary
Timeout and failure handling test/cppqualcheck.sh	The script selects timeouts based on binary flavour, records command failures, and fails the final gate if a call fails. Probe, edit-check, and trace invocations use the configured timeout.
Cached repo-root checks test/cppqualcheck.sh	Repo-root checks use a shared temporary cache with one file per verb. Checks preserve command status, and cleanup removes the cache directory.

Priority: ⬇️ Low

Estimated code review effort: 3 (Moderate) | ~20 minutes

Change: Other

Suggested reviewers: barefootski, laimawu

Merge Risk: ⚪ Minimal · up to 00c30

This change updates documentation, comments and test-gate behavior, and does not alter product runtime behavior. No merge-blocking risk is evident.

Pre-merge checks · 4

Passed checks (4 passed)

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title accurately summarizes the main changes: cppqualcheck timeout reporting, review-note updates, and CONTRIBUTING map changes. It is specific and related to the changeset, although it includes s…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

Warning

Some tools did not complete. Review the errors below.

🔧 ast-grep (0.45.3)

src/graph.h

ast-grep timed out on this file

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