Skip to content

chore(docs): epic #920 closeout — update counts & add new schema objects#946

Merged
ericsocrat merged 1 commit intomainfrom
chore/epic-920-closeout
Mar 17, 2026
Merged

chore(docs): epic #920 closeout — update counts & add new schema objects#946
ericsocrat merged 1 commit intomainfrom
chore/epic-920-closeout

Conversation

@ericsocrat
Copy link
Owner

Summary

Epic #920 (Country-Aware Scanner & Submission Pipeline) closeout — reconcile all stale documentation references after shipping 12 issues across PRs #933-#945.

Changes

CURRENT_STATE.md

copilot-instructions.md

  • QA checks: 756→759 (12 locations), view_consistency 13→16 (2 locations), total 768→771
  • Migrations: 203→227 (§3 + §7)
  • Products: 2,366→2,602, deprecated 273→58 (header)
  • §4 Key Functions: added \gs1_country_hint()\ after \is_valid_ean()\
  • §4 Views: added \�_cross_country_scan_analytics\, \�_cross_country_ean_candidates\, \�_submission_country_analytics\
  • Audit template: migration target ≥199→≥227, product target ≥2,366→≥2,602

README.md

  • Badge: QA 747→759, products 2,438→2,602
  • Numbers at a Glance: migrations 199→227, QA 747→759, products 2,438→2,602
  • Project layout tree: migrations 199→227, QA 747→759
  • CI description: QA 747→759

CHANGELOG.md

Verification

Documentation-only change (no code, no tests, no schema). CI gates (typecheck, lint, build, unit tests, Playwright) will confirm no regressions.

- CURRENT_STATE.md: update header, SHA, open PRs, add epic #920 shipped table,
  update CI gate status, key metrics, remove duplicate sections
- copilot-instructions.md: QA 756→759, view_consistency 13→16, total 768→771,
  migrations 203→227, products 2366→2602, deprecated 273→58,
  add gs1_country_hint function + 3 cross-country analytics views to §4
- README.md: QA 747→759, migrations 199→227, products 2438→2602
- CHANGELOG.md: add epic #920 entry under [Unreleased]
Copilot AI review requested due to automatic review settings March 17, 2026 19:22
@vercel
Copy link

vercel bot commented Mar 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
tryvit Ready Ready Preview, Comment Mar 17, 2026 7:24pm

@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Documentation closeout for Epic #920 (Country-Aware Scanner & Submission Pipeline), updating project-wide “current state” and reference docs to reflect recently shipped schema/QA changes and updated metrics.

Changes:

  • Updated key metrics/counters across docs (products, migrations, QA checks, Vitest counts) and added Epic #920 shipped summary.
  • Extended Copilot instructions to include new schema objects (gs1_country_hint(), cross-country analytics views) and refreshed QA/view-consistency numbers.
  • Added an Epic #920 entry to the changelog under [Unreleased].

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
copilot-instructions.md Refreshes repository “facts” (counts), adds newly shipped function/views, updates QA/view-consistency references.
README.md Updates top-level badges, “numbers at a glance”, and CI/testing counts and layout notes.
CURRENT_STATE.md Updates “last updated”, main SHA, CI gate notes, and adds an Epic #920 shipped table + metrics.
CHANGELOG.md Adds an [Unreleased] “Added” entry summarizing Epic #920 deliverables.
Comments suppressed due to low confidence (1)

README.md:309

  • This section still labels the frontend as Next.js 15 App Router, but frontend/package.json specifies Next 16.1.6. Please update the README here to match the actual major version to avoid confusion when following Next.js docs/migration guides.
│   ├── migrations/                  # 227 append-only schema migrations
│   ├── seed/                        # Reference data seeds
│   ├── tests/                       # pgTAP integration tests
│   └── functions/                   # Edge Functions (API gateway, push notifications, CAPTCHA)
│
├── frontend/                        # Next.js 15 App Router
│   ├── src/

@@ -1,15 +1,15 @@
# Copilot Instructions — TryVit

> **Last updated:** 2026-03-05
> **Ingredient analytics:** 5,340 unique ingredients (all clean ASCII English), 2,691 allergen declarations, 2,702 trace declarations
> **Ingredient concerns:** EFSA-based 4-tier additive classification (0=none, 1=low, 2=moderate, 3=high)
> **QA:** 756 checks across 48 suites + 23 negative validation tests — all passing
> **QA:** 759 checks across 48 suites + 23 negative validation tests — all passing
| **Negative Validation** | `TEST__negative_checks.sql` | 23 | Yes |

**Run:** `.\RUN_QA.ps1` — expects **768/768 checks passing** (+ EAN validation).
**Run:** `.\RUN_QA.ps1` — expects **771/771 checks passing** (+ EAN validation).

# ── Testing ──
.\RUN_QA.ps1 # 747 QA checks across 48 suites
.\RUN_QA.ps1 # 759 QA checks across 48 suites
@ericsocrat ericsocrat merged commit 6a2800b into main Mar 17, 2026
17 checks passed
@ericsocrat ericsocrat deleted the chore/epic-920-closeout branch March 17, 2026 19:39
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.

2 participants