erik/slugkit
1
0
Fork 0
Code Issues 6 Pull requests Projects Releases 6 Packages Wiki Activity Actions

Include dev environment in slug init #200

Merged
erik merged 1 commit from feat/task-634b7185-slug-init-dev-environment into main 2026-06-24 13:07:42 -05:00
Conversation 2 Commits 1 Files changed 12 +155 -19
erik commented 2026-06-24 09:58:38 -05:00
Owner
Copy link

Summary

Add generated-site dev environment scaffolding so slug init creates an Overmind-based local development setup.

Task

Task: #task-634b7185

Changes

  • Added template-site Makefile targets for starting, stopping, checking, and connecting to the Overmind dev environment.
  • Added template-site Procfile.dev that runs the Hono app and Tailwind watcher from a standalone generated site.
  • Updated template setup docs to make make dev the generated-site local development path.
  • Made template-site test and TypeScript config standalone-safe by removing repository-relative config paths.
  • Updated slug init tests to verify generated dev environment files and standalone config.

Testing

  • Unit tests added/updated
  • Manual testing performed

Verification:

  • npm exec --workspace @evcraddock/slug-cli -- vitest run --config ../vitest.config.ts src/__tests__/commands.test.ts
  • npm run build --workspace @slugkit/template-site && npm test --workspace @slugkit/template-site
  • ./scripts/pre-pr.sh

Checklist

  • ./scripts/pre-pr.sh passes
  • Documentation updated
  • No unrelated changes included
## Summary Add generated-site dev environment scaffolding so `slug init` creates an Overmind-based local development setup. ## Task Task: #task-634b7185 ## Changes - Added template-site `Makefile` targets for starting, stopping, checking, and connecting to the Overmind dev environment. - Added template-site `Procfile.dev` that runs the Hono app and Tailwind watcher from a standalone generated site. - Updated template setup docs to make `make dev` the generated-site local development path. - Made template-site test and TypeScript config standalone-safe by removing repository-relative config paths. - Updated `slug init` tests to verify generated dev environment files and standalone config. ## Testing - [x] Unit tests added/updated - [x] Manual testing performed Verification: - `npm exec --workspace @evcraddock/slug-cli -- vitest run --config ../vitest.config.ts src/__tests__/commands.test.ts` - `npm run build --workspace @slugkit/template-site && npm test --workspace @slugkit/template-site` - `./scripts/pre-pr.sh` ## Checklist - [x] `./scripts/pre-pr.sh` passes - [x] Documentation updated - [x] No unrelated changes included
feat: include dev environment in slug init
All checks were successful
CI / build-lint-test (pull_request) Successful in 48s
Details
566392bb90 
Task: #task-634b7185
erik commented 2026-06-24 10:00:27 -05:00
Author
Owner
Copy link

PR review completed for task task-634b7185.

Result: approved

Acceptance criteria: 3/3 met, 0 missing.

Verification:

  • Reviewed task scope, PR body, and diff for generated-site Overmind dev environment support.
  • Confirmed slug init output now includes Makefile and Procfile.dev from the template.
  • Confirmed generated dev commands are standalone-site appropriate and avoid repository template/site paths.
  • Confirmed template package scripts and tsconfig.json no longer depend on repository-relative config paths, so generated sites are more suitable for local website changes.
  • Local targeted CLI test passed: npm exec --workspace @evcraddock/slug-cli -- vitest run --config ../vitest.config.ts src/__tests__/commands.test.ts.
  • Local template verification passed: npm run build --workspace @slugkit/template-site && npm test --workspace @slugkit/template-site.
  • Local full gate passed: ./scripts/pre-pr.sh.
  • Forgejo Actions #280 succeeded for commit 566392b (build-lint-test, pull_request).

Blocking issues: none.
Warnings: none.

Merge gate: waiting for explicit human approval.

PR Review Worker Result

  • pr: open
  • review: approved
  • pr_comment: posted
  • task_comment: posted
  • blocking_issues: 0
  • warnings: 0
  • criteria_total: 3
  • criteria_met: 3
  • criteria_missing: 0
PR review completed for task `task-634b7185`. Result: approved Acceptance criteria: 3/3 met, 0 missing. Verification: - Reviewed task scope, PR body, and diff for generated-site Overmind dev environment support. - Confirmed `slug init` output now includes `Makefile` and `Procfile.dev` from the template. - Confirmed generated dev commands are standalone-site appropriate and avoid repository `template/site` paths. - Confirmed template package scripts and `tsconfig.json` no longer depend on repository-relative config paths, so generated sites are more suitable for local website changes. - Local targeted CLI test passed: `npm exec --workspace @evcraddock/slug-cli -- vitest run --config ../vitest.config.ts src/__tests__/commands.test.ts`. - Local template verification passed: `npm run build --workspace @slugkit/template-site && npm test --workspace @slugkit/template-site`. - Local full gate passed: `./scripts/pre-pr.sh`. - Forgejo Actions #280 succeeded for commit `566392b` (`build-lint-test`, pull_request). Blocking issues: none. Warnings: none. Merge gate: waiting for explicit human approval. PR Review Worker Result - pr: open - review: approved - pr_comment: posted - task_comment: posted - blocking_issues: 0 - warnings: 0 - criteria_total: 3 - criteria_met: 3 - criteria_missing: 0
erik force-pushed feat/task-634b7185-slug-init-dev-environment from 566392bb90
All checks were successful
CI / build-lint-test (pull_request) Successful in 48s
Details
to d471b8600d
All checks were successful
CI / build-lint-test (pull_request) Successful in 52s
Details
2026-06-24 13:02:44 -05:00 Compare
erik commented 2026-06-24 13:04:35 -05:00
Author
Owner
Copy link

PR review completed for task task-634b7185 after update d471b86.

Result: approved

Acceptance criteria: 3/3 met, 0 missing.

Verification:

  • Reviewed updated PR diff and commit d471b86.
  • Confirmed slug init output includes generated-site Makefile, Procfile.dev, and standalone vitest.config.ts.
  • Confirmed generated dev commands are standalone-site appropriate and avoid repository template/site paths.
  • Confirmed generated sites deliberately exclude toolkit-only tests while Slugkit itself keeps them:
    • src/__tests__/dev-environment.test.ts
    • src/federation/__tests__/follow.test.ts
  • Confirmed generated sites include standalone TypeScript/Vitest dev dependencies and no longer inherit repository-relative TypeScript/Vitest config.
  • Local targeted CLI/template tests passed: npm exec --workspace @evcraddock/slug-cli -- vitest run --config ../vitest.config.ts src/__tests__/commands.test.ts src/__tests__/template-package-script.test.ts and npm exec --workspace @slugkit/template-site -- vitest run --config ./vitest.config.ts src/__tests__/dev-environment.test.ts src/federation/__tests__/follow.test.ts src/api/__tests__/comments.test.ts.
  • Manual generated-site smoke passed at /Users/erik/Private/code/forgejo/my-site: npm install, npm run db:migrate, npm run build, npm test (41 files, 263 tests), and make dev/make dev-stop.
  • Local full gate passed: ./scripts/pre-pr.sh.
  • Forgejo Actions #281 succeeded for commit d471b86 (build-lint-test, pull_request).

Blocking issues: none.
Warnings: none.

Merge gate: waiting for explicit human approval.

PR Review Worker Result

  • pr: open
  • review: approved
  • pr_comment: posted
  • task_comment: posted
  • blocking_issues: 0
  • warnings: 0
  • criteria_total: 3
  • criteria_met: 3
  • criteria_missing: 0
PR review completed for task `task-634b7185` after update `d471b86`. Result: approved Acceptance criteria: 3/3 met, 0 missing. Verification: - Reviewed updated PR diff and commit `d471b86`. - Confirmed `slug init` output includes generated-site `Makefile`, `Procfile.dev`, and standalone `vitest.config.ts`. - Confirmed generated dev commands are standalone-site appropriate and avoid repository `template/site` paths. - Confirmed generated sites deliberately exclude toolkit-only tests while Slugkit itself keeps them: - `src/__tests__/dev-environment.test.ts` - `src/federation/__tests__/follow.test.ts` - Confirmed generated sites include standalone TypeScript/Vitest dev dependencies and no longer inherit repository-relative TypeScript/Vitest config. - Local targeted CLI/template tests passed: `npm exec --workspace @evcraddock/slug-cli -- vitest run --config ../vitest.config.ts src/__tests__/commands.test.ts src/__tests__/template-package-script.test.ts` and `npm exec --workspace @slugkit/template-site -- vitest run --config ./vitest.config.ts src/__tests__/dev-environment.test.ts src/federation/__tests__/follow.test.ts src/api/__tests__/comments.test.ts`. - Manual generated-site smoke passed at `/Users/erik/Private/code/forgejo/my-site`: `npm install`, `npm run db:migrate`, `npm run build`, `npm test` (41 files, 263 tests), and `make dev`/`make dev-stop`. - Local full gate passed: `./scripts/pre-pr.sh`. - Forgejo Actions #281 succeeded for commit `d471b86` (`build-lint-test`, pull_request). Blocking issues: none. Warnings: none. Merge gate: waiting for explicit human approval. PR Review Worker Result - pr: open - review: approved - pr_comment: posted - task_comment: posted - blocking_issues: 0 - warnings: 0 - criteria_total: 3 - criteria_met: 3 - criteria_missing: 0
erik merged commit 1ba730e619 into main 2026-06-24 13:07:42 -05:00
erik deleted branch feat/task-634b7185-slug-init-dev-environment 2026-06-24 13:07:42 -05:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
activitypub

   
admin

   
api

   
articles

   
auth

   
bug

   
cleanup

   
cli

   
comments

   
compatibility

   
config

   
contacts

   
database

   
deployment

   
design

   
dev-env

   
docs

   
documentation

   
email

   
enhancement

   
feature

   
federation

   
feed

   
homepage

   
implementation

   
integration

   
media

   
openapi

   
priority:high

   
priority:low

   
priority:medium

   
proof

   
public-routes

   
public-ui

   
release

   
safety

   
social

   
sources

   
status:active

   
status:canceled

   
status:done

   
status:inprogress

   
status:waiting

   
syndication

   
tailwind

   
template

   
test

   
web
No labels
activitypub
admin
api
articles
auth
bug
cleanup
cli
comments
compatibility
config
contacts
database
deployment
design
dev-env
docs
documentation
email
enhancement
feature
federation
feed
homepage
implementation
integration
media
openapi
priority:high
priority:low
priority:medium
proof
public-routes
public-ui
release
safety
social
sources
status:active
status:canceled
status:done
status:inprogress
status:waiting
syndication
tailwind
template
test
web
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
erik/slugkit!200
No description provided.