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

None yet

Development

Successfully merging this pull request may close these issues.

1 participant