Skip to content

PR_26180_OWNER_009-move-api-application#288

Draft
ToolboxAid wants to merge 1 commit into
PR_26180_OWNER_008-move-www-applicationfrom
PR_26180_OWNER_009-move-api-application
Draft

PR_26180_OWNER_009-move-api-application#288
ToolboxAid wants to merge 1 commit into
PR_26180_OWNER_008-move-www-applicationfrom
PR_26180_OWNER_009-move-api-application

Conversation

@ToolboxAid

Copy link
Copy Markdown
Owner

Summary

Moves server/API application runtime into top-level api/ while preserving /api/* route behavior.

  • Moves Local API server/router/services/persistence/storage/auth/seed/admin server modules under api/.
  • Keeps developer-only local bootstrap/orchestration in dev/ for the later local-runtime PR.
  • Keeps browser API clients in src/api/ so www/ does not import top-level api/ implementation files directly.
  • Updates Project Instructions version to 2026.06.28.009 and backlog status for Repository Architecture Simplification.

Stacked Dependency

Base branch: PR_26180_OWNER_008-move-www-application

Validation

  • PASS: git diff --check
  • PASS: npm run validate:canonical-structure
  • PASS: node --check for 86 changed JS/MJS files
  • PASS: targeted Local API/server node tests, 7/7 files and 30/30 subtests
  • PASS: inline /api/* route smoke for representative provider-independent routes
  • PASS: npx playwright test dev/tests/playwright/tools/BrowserApiUrlConfig.spec.mjs --reporter=line

Reports And Artifact

Reports are under dev/reports/:

  • codex_review.diff
  • codex_changed_files.txt
  • PR_26180_OWNER_009-move-api-application_report.md
  • PR_26180_OWNER_009-move-api-application_branch-validation.md
  • PR_26180_OWNER_009-move-api-application_requirement-checklist.md
  • PR_26180_OWNER_009-move-api-application_validation-report.md
  • PR_26180_OWNER_009-move-api-application_manual-validation-notes.md

Outcome ZIP exists locally under dev/workspace/zips/PR_26180_OWNER_009-move-api-application_delta.zip.

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