Skip to content

fix: close BUG-022..027, BUG-029, BUG-031, BUG-033 from QA sweep - #41

Open
amar-python wants to merge 7 commits into
mainfrom
claude/csv-loading-postgres-migration-qvkl4l
Open

fix: close BUG-022..027, BUG-029, BUG-031, BUG-033 from QA sweep#41
amar-python wants to merge 7 commits into
mainfrom
claude/csv-loading-postgres-migration-qvkl4l

Conversation

@amar-python

Copy link
Copy Markdown
Owner

No description provided.

@amar-python amar-python closed this Aug 2, 2026
@amar-python amar-python reopened this Aug 2, 2026
amar-python pushed a commit that referenced this pull request Aug 3, 2026
…RT.md claims

BUG_REPORT.md on main claimed 34/34 RESOLVED, but the code fixes for the
above eight bugs only lived on the abandoned PR #41 branch. This commit
brings main's code in line with the report:

- dynamic_loader.py: \x1f row-hash separator (BUG-022) + MAX_ROWS guard +
  MAX_ROW_ERRORS_REPORTED cap (BUG-027)
- __root.tsx: replace Lovable branding with CSV Migrator meta/OG tags (BUG-024)
- csv_routes.py: try/except around /preview parse path (BUG-025)
- lovable-error-reporting.ts: ANALYTICS_ENABLED gate, default off (BUG-029)
- main.py: /api/health runs LIMIT 0 probe on uploads schema (BUG-031)
- db.py: init_pool exponential-backoff retry, layered onto ThreadedConnectionPool (BUG-026)
- quality-gate.yml: add frontend-build (Node 20 + npm ci + lint + build) (BUG-023)
- test_api.py: row-hash split-boundary + still-dedupes regressions (BUG-022)
- test_ci_quality_gate.py: split PYTHON_JOBS / NODE_JOBS so frontend-build passes structure checks
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