Skip to content

CCM-14750: Add release-check commit mappings - #301

Draft
m-houston wants to merge 4 commits into
ccm-14750/release-check-report-rollupfrom
ccm-14750/release-check-commit-mappings
Draft

m-houston wants to merge 4 commits into
ccm-14750/release-check-report-rollupfrom
ccm-14750/release-check-commit-mappings

Conversation

@m-houston

Copy link
Copy Markdown

Description

This slice adds commit-to-ticket mapping support for cases where a PR or commit carries the wrong Jira key.

It includes:

  • support for loading commit-to-ticket overrides from a mapping file
  • automatic detection of a .jira-commits file in the target repository root, with CLI override support
  • comparison and fix-flow updates so mapped tickets are used consistently instead of the detected key
  • reporting updates so mapped commits are visible and the mapped/detected ticket relationship is clear
  • tests covering mapping-file parsing, override application, report rendering, and branch-specific validation

Context

NHS governance checks depend on the tool being able to represent the intended Jira ticket for the code being released. In practice, some PRs and commits carry the wrong issue number, which would otherwise make the governance report and any follow-up fixes misleading.

This slice makes the release-check tooling resilient to that data-quality problem by allowing explicit commit-level Jira overrides while keeping those overrides visible in the report output.

Why this is a separate slice

Although it touches reporting, the main concern here is semantic correction of commit-to-ticket matching. Splitting it from the surrounding roll-up work keeps the review focused on the new mapping contract, how it affects comparison/fix behaviour, and how the override is surfaced to operators.

Validation

  • package install
  • lint
  • typecheck
  • unit tests

Sensitive Information Declaration

  • I confirm that neither PII/PID nor sensitive data are included in this PR and the codebase changes.

@m-houston
m-houston added this pull request to stack #299 September 25, 2026 08:20
@m-houston
m-houston force-pushed the ccm-14750/release-check-commit-mappings branch from 294c02a to 852c6dc Compare September 25, 2026 08:22
m-houston and others added 4 commits September 25, 2026 17:46
Keep mapped commit reporting easier to review by showing the mapped and detected tickets side by side.

Co-authored-by: Copilot <[email protected]>
Keep the resliced mapping branch lint-clean and restore coverage after separating it from the report-rollup slice.

Co-authored-by: Copilot <[email protected]>
Allow repository-wide mapping files to include commits outside the selected release range without failing the run.

Co-authored-by: Copilot <[email protected]>
@m-houston
m-houston force-pushed the ccm-14750/release-check-commit-mappings branch from 852c6dc to 5f5b11f Compare September 25, 2026 16:48

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant