Repository navigation
docs(copy): remove EDI and SFTP from supported-format copy - #3570
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
No actionable comments were generated in the recent review.
|
PetrefiedThunder
left a comment
There was a problem hiding this comment.
Reviewed head 06c7476. Removes EDI/EDI-style from supported-format copy in README, REPO_PURPOSE, PRODUCTION_SURFACE, PRODUCT_DESCRIPTION, WEBSITE_COPY and drops the EDI source label on /records. Repo-wide grep of frontend/src, frontend/public and docs at this head finds no remaining user-facing EDI/SFTP supported-format claims: remaining hits are internal ids (csv_sftp_import / csv-sftp, both rendered as 'CSV Import'), the IntegrationProfileSourceType union, capabilities.ts neverShown list, and research/audit/security/QA-artifact docs. No sample or invented data added. Nit (non-blocking): any legacy record with source_system 'edi' now renders the raw key 'edi' via the fallback at records/page.tsx:148,237. CI was still pending when reviewed (5 pass, 28 pending); merge needs green CI.
REVIEW: PASS codex 06c7476
✅ Bundle Size Report
Bundle is within the 10.0MB limit. |
PR Blocker ChecklistHead SHA: Status: Ready
|
|
Vercel preview access note removed (A-2177 secret-hygiene cleanup). |
Refs SEL-450, SEL-439, SEL-443.
edilabel on the records page.csv-sftp, and thecapabilities.neverShownguard.Verification: tsc, lint, knip and vitest were not run locally because host load was above the 28 threshold. CI is the gate.
Note
Low Risk
Documentation and a display-label mapping only; no runtime ingestion, auth, or persistence behavior changes.
Overview
Aligns product and engineering docs with webhook, CSV, and EPCIS as the stated FSMA intake surfaces by stripping EDI (and EDI-style wording on the website) from
PRODUCTION_SURFACE.md,README.md,REPO_PURPOSE.md,docs/PRODUCT_DESCRIPTION.md, anddocs/product/WEBSITE_COPY.md. Capability tables, production-spine narrative, GS1 validation copy, and ingestion service descriptions no longer list EDI.On the dashboard,
frontend/src/app/records/page.tsxdrops theedientry fromSOURCE_LABEL, so EDI-sourced records show the rawsource_systemvalue instead of “EDI”. No ingestion APIs, routers, or EDI implementation code are removed in this diff.Reviewed by Cursor Bugbot for commit 06c7476. Bugbot is set up for automated code reviews on this repo. Configure here.