Skip to content

perf(integrity): scope the PR integrity gate to changed records - #119

Merged
Seungpyo1007 merged 1 commit into
mainfrom
perf/integrity-scoped
Sep 30, 2026
Merged

Seungpyo1007 merged 1 commit into
mainfrom
perf/integrity-scoped

Conversation

@Seungpyo1007

Copy link
Copy Markdown
Member

Step 8 of GetTechAPI/TechAPI#350. Removes the last full-tree scans from the PR path.

  • integrity_check.py --only FILE: hard anomalies (verified-without-sources, slug!=file, single>multi) for listed paths; duplicate slug/name against the full catalog (file-name index; cpu/gpu name index).
  • Advisory statistical sections (cross-source ratios) are skipped when scoped; the weekly refresh still runs the full scan.
  • PR validation comment: builds changed-paths.txt from the merge-base diff and runs head and base with it.
  • Checked on a synthetic tree: unchanged-file anomalies ignored, changed-file dup slug/name and verified-without-sources caught.

`integrity_check.py --only FILE` checks just the listed data-relative paths:
per-record hard anomalies, plus duplicate slug/name against the whole
catalog via file names (and a full cpu/gpu name index). The population-based
advisory sections are skipped when scoped. The PR validation comment runs it
on head and base with the same changed-path list; the weekly refresh still
runs the full scan.

Refs GetTechAPI/TechAPI#350
@Seungpyo1007 Seungpyo1007 added the ci Continuous integration changes label Sep 30, 2026
@Seungpyo1007 Seungpyo1007 self-assigned this Sep 30, 2026
@Seungpyo1007
Seungpyo1007 merged commit 0044ab2 into main Sep 30, 2026
1 check passed
@Seungpyo1007
Seungpyo1007 deleted the perf/integrity-scoped branch September 30, 2026 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Continuous integration changes

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant