docs: update GAP_ANALYSIS.md for current state #28

New Issue

forbes · 2026-02-08T19:51:19Z

forbes commented

2026-02-08 19:51:19 +00:00

Problem

GAP_ANALYSIS.md references non-existent files, has stale claims, and ignores the React frontend entirely.

Specific Issues

Section 1.1: Lists silo-spec.md as "Consolidated" — it is a 1-line redirect stub that should be deleted
Section 1.2: Says "Configuration Reference: Open" — but docs/CONFIGURATION.md exists and is comprehensive (294 lines). Should be marked done.
Section 2.2: Shows property_schema_version column from migration 005 — 005 does not exist on disk
Section 2.3 (Phase 2.2): References docs/AUTH.md — exists, good. But says "Migration: 009_auth.sql" which is not on disk
Section 2.3 (Phase 2.3): References internal/api/handlers_audit.go — this file does not exist (handlers are in handlers.go)
Appendix A: File structure lists handlers_audit.go, 008_odoo_integration.sql, 009_auth.sql, 010_item_extended_fields.sql — none exist on disk
Appendix A: Future structure says 011_item_locks.sql — but 011 is already used for item_files
No mention of React SPA — the frontend migration is a major change not reflected
No coverage of open Gitea issues — 11+ open issues are untracked in the gap analysis
Section 5: "Documentation Overhaul" listed as medium-term — DEPLOYMENT.md and CONFIGURATION.md are already done

Acceptance Criteria

Remove reference to silo-spec.md
Mark Configuration Reference as done
Fix all references to non-existent files
Update Appendix A file structure to match reality
Add React frontend section or note
Fix future migration numbering (011 is taken)
Update Section 5 recommended order for completed items

## Problem GAP_ANALYSIS.md references non-existent files, has stale claims, and ignores the React frontend entirely. ## Specific Issues 1. **Section 1.1**: Lists `silo-spec.md` as "Consolidated" — it is a 1-line redirect stub that should be deleted 2. **Section 1.2**: Says "Configuration Reference: Open" — but `docs/CONFIGURATION.md` exists and is comprehensive (294 lines). Should be marked done. 3. **Section 2.2**: Shows `property_schema_version` column from migration 005 — 005 does not exist on disk 4. **Section 2.3 (Phase 2.2)**: References `docs/AUTH.md` — exists, good. But says "Migration: 009_auth.sql" which is not on disk 5. **Section 2.3 (Phase 2.3)**: References `internal/api/handlers_audit.go` — this file does not exist (handlers are in `handlers.go`) 6. **Appendix A**: File structure lists `handlers_audit.go`, `008_odoo_integration.sql`, `009_auth.sql`, `010_item_extended_fields.sql` — none exist on disk 7. **Appendix A**: Future structure says `011_item_locks.sql` — but 011 is already used for `item_files` 8. **No mention of React SPA** — the frontend migration is a major change not reflected 9. **No coverage of open Gitea issues** — 11+ open issues are untracked in the gap analysis 10. **Section 5**: "Documentation Overhaul" listed as medium-term — DEPLOYMENT.md and CONFIGURATION.md are already done ## Acceptance Criteria - [ ] Remove reference to `silo-spec.md` - [ ] Mark Configuration Reference as done - [ ] Fix all references to non-existent files - [ ] Update Appendix A file structure to match reality - [ ] Add React frontend section or note - [ ] Fix future migration numbering (011 is taken) - [ ] Update Section 5 recommended order for completed items

forbes referenced this issue from a commit

2026-02-08 20:11:27 +00:00

docs: update GAP_ANALYSIS.md for current state

forbes referenced this issue

2026-02-08 20:11:39 +00:00

docs: update all documentation for React SPA migration #32

forbes closed this issue

2026-02-08 20:13:38 +00:00

Sign in to join this conversation.