ci(action): Cover lock-risk caution path and document dialect input

[mhiro2]

Summary

• Document that the dialect input now drives both the SQL parser and the lock-risk rule evaluation in mode: review, and add a dedicated "Lock-risk findings" section to action/README.md pointing users at deny: caution, except-rules, and per-rule severity_overrides.
• Add a workflow step to test-action.yaml that runs the action against fixtures/review/lock-risk/add-index-on-large-table-postgres with dialect: postgres and asserts summary-caution >= 1, locking the lock-risk wiring through the action.

Changes

• 9b31acd : docs(action): note dialect drives lock-risk rule evaluation
  • Update the DIALECT comment block in action/review.sh to call out the parser + lock-risk evaluation roles
  • Refresh the dialect input row in action/README.md and add a "Lock-risk findings" section covering activation, the caution band, and the noise-tuning escape hatches
• adea681 : ci(action): cover lock-risk caution path with dialect input
  • Add a review smoke step that exercises the action with dialect: postgres against the add-index lock-risk fixture
  • Assert has-findings=true, has-blocking-findings=false (caution stays below the default deny: breaking gate), and summary-caution >= 1

[github-actions]

Code Metrics Report

	main (1f72f53)	#128 (69c2b4e)	+/-
Coverage	94.7%	94.7%	0.0%
Test Execution Time	1m35s	1m24s	-11s

Details

  |                     | main (1f72f53) | #128 (69c2b4e) | +/-  |
  |---------------------|----------------|----------------|------|
  | Coverage            |          94.7% |          94.7% | 0.0% |
  |   Files             |             81 |             81 |    0 |
  |   Lines             |          38211 |          38211 |    0 |
  |   Covered           |          36207 |          36207 |    0 |
+ | Test Execution Time |          1m35s |          1m24s | -11s |

---

Reported by octocov

[github-actions]

Schema review

Tip

✅ No risk findings — schema changes look safe to merge.