Compare commits

...
Sign in to create a new pull request.

5 commits

Author SHA1 Message Date
copilot-swe-agent[bot]
5979296f3b Merge remote-tracking branch 'refs/remotes/origin/main' into copilot/identify-spec-implementation-drift
# Conflicts:
#	apps/planner/app/components/PlannerMap.tsx
#	apps/planner/app/lib/brouter.ts
#	openspec/specs/no-go-areas/spec.md

Co-authored-by: stigi <13815+stigi@users.noreply.github.com>
2026-04-03 09:42:51 +00:00
copilot-swe-agent[bot]
009a2f1f19
Update spec drift report after merging main: 11 of 12 drifts now resolved
Agent-Logs-Url: https://github.com/trails-cool/trails/sessions/4d97b64f-ba95-4986-8443-d731c19cc20d

Co-authored-by: stigi <13815+stigi@users.noreply.github.com>
2026-04-03 09:25:24 +00:00
copilot-swe-agent[bot]
9c3900e7b5 Merge remote-tracking branch 'origin/main' into copilot/identify-spec-implementation-drift
Co-authored-by: stigi <13815+stigi@users.noreply.github.com>
2026-04-03 09:23:17 +00:00
copilot-swe-agent[bot]
4e9e7d7cb6
Add spec drift report documenting 12 drifts between specs and implementation
Agent-Logs-Url: https://github.com/trails-cool/trails/sessions/ad77fee7-e03f-4695-86e2-1a6cb3c28977

Co-authored-by: stigi <13815+stigi@users.noreply.github.com>
2026-04-01 13:34:10 +00:00
copilot-swe-agent[bot]
89788a6ab7
Initial plan 2026-04-01 13:06:24 +00:00

32
docs/spec-drift.md Normal file
View file

@ -0,0 +1,32 @@
# Spec ↔ Implementation Drift Report
This document lists all known drifts between the OpenSpec specifications in `openspec/specs/` and the actual implementation.
> **Last updated**: 2026-04-03
---
## Active Drifts
None — all identified drifts have been resolved.
---
## Resolved Drifts
Previously identified drifts that have been fixed via code changes or spec updates.
| # | Drift | Fixed in | Resolution |
|---|-------|----------|------------|
| 1 | PostGIS `geom` column never populated | PR #150, #151 | `setGeomFromGpx()` now populates geometry via `ST_GeomFromGeoJSON` on route/activity creation |
| 2 | Planner metrics gauges always zero | PR #149 | Gauges wired to `.inc()`/`.dec()` in `yjs-server.ts` on client connect/disconnect |
| 3 | Email provider spec said Resend, impl uses SMTP | PR #153 | Spec updated to document Nodemailer + SMTP |
| 4 | No-go areas converted to circles instead of polygons | PR #155 | `noGoAreasToParam()` now passes polygon vertices directly via BRouter's `polygons` parameter |
| 5 | Server type spec said CX21, Terraform has cx23 | PR #153 | Spec updated to document cx23 |
| 6 | Secret files split into two instead of one | PR #153 | Spec updated to document `secrets.app.env` + `secrets.infra.env` split |
| 7 | BRouter host failover spec said "within 5 seconds" | PR #153 | Spec updated to document immediate deterministic election |
| 8 | Health endpoint extra `version` field | PR #153 | Spec updated to include `version` in response format |
| 9 | Extra `brouter_request_duration_seconds` metric | PR #153 | Spec updated to document the metric |
| 10 | 5 routing profiles vs 2 in spec | PR #153 | Spec updated to list all 5 profiles |
| 11 | Session 30-day max age ceiling not enforced | PR #153 | Spec updated to document no hard ceiling |
| 12 | Interactive map features in planner app, not shared package | PR #153 | Spec updated to document architecture decision |