Build template repository foundation #39

Merged
erik merged 9 commits from feat/task-d6397b57-template-foundation into main 2026-05-19 20:31:08 -05:00
Owner

Summary

  • Move the Hono template app into template/site so it can be copied by future slug init work.
  • Add template-site package metadata, docs, environment example, route/template structure, and placeholder directories for future specs.
  • Add Forgejo CI for build, lint, and tests on PRs and main pushes.
  • Update root scripts/README/Makefile to run the in-repo template through npm workspaces.

Testing

  • npm run build
  • make check
  • make dev + curl http://localhost:3000/health + homepage check
  • make pre-pr

Task: #task-d6397b57

## Summary - Move the Hono template app into `template/site` so it can be copied by future `slug init` work. - Add template-site package metadata, docs, environment example, route/template structure, and placeholder directories for future specs. - Add Forgejo CI for build, lint, and tests on PRs and main pushes. - Update root scripts/README/Makefile to run the in-repo template through npm workspaces. ## Testing - `npm run build` - `make check` - `make dev` + `curl http://localhost:3000/health` + homepage check - `make pre-pr` Task: #task-d6397b57
feat: build template site foundation
Some checks failed
CI / build-lint-test (pull_request) Failing after 2m10s
268874d896
Task: #task-d6397b57
fix: avoid external actions in CI workflow
Some checks failed
CI / build-lint-test (pull_request) Failing after 1s
9e709bd978
Task: #task-d6397b57
fix: make CI checkout robust
Some checks failed
CI / build-lint-test (pull_request) Failing after 1s
66f4afd06a
Task: #task-d6397b57
fix: trust Forgejo CA during CI checkout
Some checks failed
CI / build-lint-test (pull_request) Failing after 2s
5f1972590d
Task: #task-d6397b57
fix: install Node in CI
Some checks failed
CI / build-lint-test (pull_request) Failing after 13s
f6289e75db
Task: #task-d6397b57
fix: use current Node in CI
Some checks failed
CI / build-lint-test (pull_request) Failing after 17s
724c24197c
Task: #task-d6397b57
fix: sync lockfile with current npm
All checks were successful
CI / build-lint-test (pull_request) Successful in 26s
d8d96330d1
Task: #task-d6397b57
feat: expose dev server on LAN
All checks were successful
CI / build-lint-test (pull_request) Successful in 21s
563556e272
Task: #task-d6397b57
fix: load template env files
All checks were successful
CI / build-lint-test (pull_request) Successful in 24s
a3a19aae9d
Task: #task-d6397b57
Author
Owner

PR Review: Approved

Summary

The PR builds the in-repo Slugkit template site foundation in template/site, wires root workspace scripts to it, adds health/home routes with tests, documents template usage, and adds Forgejo CI for PRs and main pushes.

Acceptance Criteria

  • In-repo template dependencies install successfully (npm ci verified by CI).
  • Documented dev command starts the template website (make dev, with Procfile.dev targeting the template workspace).
  • GET /health returns HTTP 200 with { "status": "ok" }.
  • GET / renders a placeholder public homepage.
  • Automated tests cover health and home routes.
  • Template files are organized under template/site for future slug init copying.
  • README text explains this is the in-repo site template, not CLI source.
  • CI workflow exists for build, lint, and tests on PRs and main pushes.
  • Relevant checks pass: ./scripts/pre-pr.sh and Forgejo CI succeeded.

Blocking Issues

None.

Warnings

None.

Verdict

Approved for merge.

## PR Review: Approved ### Summary The PR builds the in-repo Slugkit template site foundation in `template/site`, wires root workspace scripts to it, adds health/home routes with tests, documents template usage, and adds Forgejo CI for PRs and `main` pushes. ### Acceptance Criteria - [x] In-repo template dependencies install successfully (`npm ci` verified by CI). - [x] Documented dev command starts the template website (`make dev`, with `Procfile.dev` targeting the template workspace). - [x] `GET /health` returns HTTP 200 with `{ "status": "ok" }`. - [x] `GET /` renders a placeholder public homepage. - [x] Automated tests cover health and home routes. - [x] Template files are organized under `template/site` for future `slug init` copying. - [x] README text explains this is the in-repo site template, not CLI source. - [x] CI workflow exists for build, lint, and tests on PRs and main pushes. - [x] Relevant checks pass: `./scripts/pre-pr.sh` and Forgejo CI succeeded. ### Blocking Issues None. ### Warnings None. ### Verdict Approved for merge.
erik merged commit 6c81a017df into main 2026-05-19 20:31:08 -05:00
Sign in to join this conversation.
No description provided.