Skip to content

Final - #9

Merged
harshada2576 merged 22 commits into
mainfrom
final
Nov 5, 2026
Merged

harshada2576 merged 22 commits into
mainfrom
final

Conversation

@harshada2576

Copy link
Copy Markdown
Owner

No description provided.

…rage-audit

Feature/step 2.11 mumbai coverage audit
Resolves Phases.md conflict: dev carried a stale tail (outdated Phase-2
recovery next-step + leftover separator + duplicated Phase-1 block from
the 61a1333 merge). Feature side is authoritative - Phase 3 verified
complete, next is Phase 4 per docs/phase_3_closure_report.md. Memory.md
auto-merged clean.

Verify: tsc clean, node suites 117/117 pass.
…ICP framework

- Secondary ingestion: OSM Overpass + data.gov.in adapters through the
  canonical pipeline; shared retry/SSRF-hardened transport; multi-source
  runner (--source) with unattended-scheduling guard (OCM cron only)
- Phase 5 ML: maturity gates, reproducible training (chronological splits,
  baseline comparison, overlap lock, ml.* recording), versioned registry
  (verify/approve/rollback), fail-closed inference, operator CLI, and
  /api/ml/status + /api/ml/predict (no UI changes; production inference
  disabled at COLD/NO_MODEL until real evidence exists)
- CPO: OCPI 2.2.1 Locations + OICP 2.3 PullEVSEData subset, partner
  lifecycle (all NOT_CONFIGURED), canonical sync with cursors, isolated
  simulator conformance tests, dry-run-first CLIs
- Docs: secondary_sources, phase5_ml, cpo_partners guides; README structure
  + test count; dated Phases/Memory notes. Env template placeholders only.
  Workflow least-privilege (contents: read).
- Tests: 603 passed (+151), 2 skipped; 7 pre-existing live-DB tests remain
  environment-blocked (placeholder DB host). No migrations, no secrets.
Merge PR #8 (phase-6-production) with the completion-mission work.

Conflict resolutions (14 overlapping paths):
- backend/ml/{__init__,gates,inference,registry}.py: keep fail-closed
  mission versions (tested; production inference stays disabled without
  real evidence). Remove backend/ml/models.py: sole consumer was the
  replaced test_phase5 and its gates paradigm is incompatible.
- tests/test_phase5.py: keep mission suite (37 tests: gates, training,
  registry, inference, CLI contracts on labelled synthetic data).
- .github/workflows/scheduled_ingestion.yml: keep staging pipeline,
  add secondary-source secret passthrough (manual/dry-run only).
- .env.example, README, Memory, Phases, runner, scheduling, base,
  adapters/__init__: keep both sides (verified coherent).
- requirements.txt: add numpy/pandas/scikit-learn pins for ML training.

Verification on merged tree: pytest 673 passed + 9 skipped, eslint
clean, tsc zero source errors, next build SUCCESS (incl. /api/ml/*).
No other branches touched.
Pass { useSimulatorInDev: isDevOrDemoAuth() } to sendOtp and
verifyOtpCode at every call-site in the UI layer (login submit,
verify submit, verify resend).

The simulator guard in auth.ts still independently rejects simulator
activation when isDevOrDemoAuth() is false, so production remains
fully protected at two independent layers.

Regression results (actual execution):
- test_auth_otp.mjs   10/10 pass
- npm test            139 pass / 0 fail / 1 skip (140 total)
- npm run typecheck   tsc --noEmit exit 0
- npm run lint        ESLint exit 0
- npm run build       Next.js 16.3.8 production build exit 0 (22 routes)
… driver utilities, and academic documentation
… error sanitization

- test: update expected migrations count to 16 and include 20261018000001_demo_provenance.sql
- lint: wrap initial loading state in microtask in WeatherBadge to satisfy react-hooks
- docs: reconcile spatial isolation methodology Q3 + 1.5 IQR threshold and distinguish DBSCAN noise vs 5km isolation
- test: add test_spatial_isolation_iqr_threshold_invariant enforcing k=1.5 outlier count (17)
- api: sanitize /api/ml/predict error response to prevent leaking internal execution paths
- reflect 201 node unit & integration tests (200 pass, 0 fail, 1 skipped)
- include full python test suite command
Copilot AI balanced review requested due to automatic review settings November 5, 2026 05:33
@vercel

vercel Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
charge-plus Ready Ready Preview Nov 5, 2026 5:34am UTC

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@harshada2576
harshada2576 merged commit 3c756e6 into main Nov 5, 2026
6 checks passed
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