feat(templates): add daemon project template #94

galligan · 2026-01-23T15:33:08Z

Dual binaries: CLI control + daemon process
start/stop/status commands
Unix socket server with health endpoint
Lock management via @outfitter/daemon

Co-Authored-By: Claude Opus 4.5 noreply@anthropic.com

galligan · 2026-01-23T15:33:30Z

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

greptile-apps · 2026-01-23T21:08:45Z

Greptile Summary

Adds comprehensive daemon project template with dual-binary architecture featuring CLI control interface and background daemon process
Implements Unix socket HTTP server with health monitoring, graceful shutdown, and lock-based single-instance management via @outfitter/daemon
Expands template system to include four project types (basic, CLI, MCP, daemon) with consistent tooling configurations and scaffolding support

Important Files Changed

Filename	Overview
`apps/outfitter/templates/daemon/src/daemon-main.ts.template`	Core daemon implementation with Unix socket server, health endpoints, lock management, and signal handling
`apps/outfitter/templates/daemon/src/cli.ts.template`	CLI control interface with start/stop/status commands, foreground/background execution modes, and HTTP-based daemon communication
`apps/outfitter/templates/cli/tsconfig.json.template`	TypeScript configuration with conflicting `noEmit: true` and output directory settings that prevents compilation
`apps/outfitter/templates/cli/src/program.ts.template`	CLI program template with redundant logging output using both structured logger and console.log

Confidence score: 4/5

This PR is mostly safe to merge but requires attention to several configuration issues and implementation details
Score lowered due to TypeScript configuration conflicts in CLI template, redundant logging patterns, and missing strict TypeScript settings across multiple templates that don't fully align with project standards documented in CLAUDE.md
Pay close attention to apps/outfitter/templates/cli/tsconfig.json.template which has conflicting compiler options, and review logging patterns in CLI program template for potential duplicate output

Sequence Diagram

sequenceDiagram
    participant User
    participant CLI as "CLI ({{binName}})"
    participant DaemonProcess as "Daemon Process"
    participant LockSystem as "Lock System"
    participant UnixSocket as "Unix Socket Server"
    participant FileSystem as "File System"

    User->>CLI: "{{binName}} start"
    CLI->>LockSystem: "isDaemonAlive(lockPath)"
    LockSystem-->>CLI: "false"
    CLI->>DaemonProcess: "spawn daemon process"
    DaemonProcess->>FileSystem: "mkdir daemonDir"
    DaemonProcess->>LockSystem: "acquireDaemonLock(lockPath)"
    LockSystem-->>DaemonProcess: "lock acquired"
    DaemonProcess->>UnixSocket: "Bun.serve({ unix: socketPath })"
    UnixSocket-->>DaemonProcess: "server started"
    DaemonProcess->>DaemonProcess: "setup signal handlers"
    CLI-->>User: "Daemon started"

    User->>CLI: "{{binName}} status"
    CLI->>LockSystem: "isDaemonAlive(lockPath)"
    LockSystem-->>CLI: "true"
    CLI->>UnixSocket: "fetch /health"
    UnixSocket-->>CLI: "health info"
    CLI-->>User: "status and health data"

    User->>CLI: "{{binName}} stop"
    CLI->>LockSystem: "isDaemonAlive(lockPath)"
    LockSystem-->>CLI: "true"
    CLI->>UnixSocket: "POST /shutdown"
    UnixSocket->>DaemonProcess: "shutdown signal"
    DaemonProcess->>LockSystem: "releaseDaemonLock(lock)"
    DaemonProcess->>UnixSocket: "server.stop()"
    DaemonProcess->>DaemonProcess: "process.exit(0)"
    CLI-->>User: "Daemon stopped"

greptile-apps

_{40 files reviewed, 10 comments}

_{Edit Code Review Agent Settings | Greptile}

greptile-apps · 2026-01-23T21:08:40Z

apps/outfitter/templates/cli/src/program.ts.template

+			logger.info`Hello, ${name}!`;
+			console.log(`Hello, ${name}!`);


style: Both structured logging and console.log output the same message, which may result in duplicate output. Should the console.log be removed since structured logging is already handling the output?

_{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}

Prompt To Fix With AI

This is a comment left during a code review. Path: apps/outfitter/templates/cli/src/program.ts.template Line: 21:22 Comment: **style:** Both structured logging and console.log output the same message, which may result in duplicate output. Should the console.log be removed since structured logging is already handling the output? Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time! How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2026-01-23T21:08:40Z

apps/outfitter/templates/cli/tsconfig.json.template

+		"noEmit": true,
+		"declaration": true,
+		"declarationDir": "dist",
+		"outDir": "dist",


logic: Configuration conflict: noEmit: true prevents TypeScript from generating any files, but declaration, declarationDir, and outDir are set to produce outputs. Either remove noEmit to allow file generation or remove the output directory settings.

Suggested change

"noEmit": true,

"declaration": true,

"declarationDir": "dist",

"outDir": "dist",

"declaration": true,

"declarationDir": "dist",

"outDir": "dist",

Prompt To Fix With AI

This is a comment left during a code review. Path: apps/outfitter/templates/cli/tsconfig.json.template Line: 9:12 Comment: **logic:** Configuration conflict: `noEmit: true` prevents TypeScript from generating any files, but `declaration`, `declarationDir`, and `outDir` are set to produce outputs. Either remove `noEmit` to allow file generation or remove the output directory settings. ```suggestion "declaration": true, "declarationDir": "dist", "outDir": "dist", ``` How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2026-01-23T21:08:41Z

apps/outfitter/templates/basic/tsconfig.json.template

+		"strict": true,
+		"noImplicitAny": true,
+		"strictNullChecks": true,
+		"noUncheckedIndexedAccess": true,
+		"noImplicitReturns": true,
+		"noFallthroughCasesInSwitch": true,


style: Missing exactOptionalPropertyTypes and noPropertyAccessFromIndexSignature settings mentioned in CLAUDE.md development principles

Suggested change

"strict": true,

"noImplicitAny": true,

"strictNullChecks": true,

"noUncheckedIndexedAccess": true,

"noImplicitReturns": true,

"noFallthroughCasesInSwitch": true,

"strict": true,

"noImplicitAny": true,

"strictNullChecks": true,

"noUncheckedIndexedAccess": true,

"exactOptionalPropertyTypes": true,

"noPropertyAccessFromIndexSignature": true,

"noImplicitReturns": true,

"noFallthroughCasesInSwitch": true,

Context Used: Context from dashboard - CLAUDE.md (source)

_{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}

Prompt To Fix With AI

This is a comment left during a code review. Path: apps/outfitter/templates/basic/tsconfig.json.template Line: 10:15 Comment: **style:** Missing `exactOptionalPropertyTypes` and `noPropertyAccessFromIndexSignature` settings mentioned in CLAUDE.md development principles ```suggestion "strict": true, "noImplicitAny": true, "strictNullChecks": true, "noUncheckedIndexedAccess": true, "exactOptionalPropertyTypes": true, "noPropertyAccessFromIndexSignature": true, "noImplicitReturns": true, "noFallthroughCasesInSwitch": true, ``` **Context Used:** Context from `dashboard` - CLAUDE.md ([source](https://app.greptile.com/review/custom-context?memory=72baf8a2-698b-40bf-966d-5b799d03ad3b)) Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time! How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2026-01-23T21:08:41Z

templates/daemon/tsconfig.json.template

+		"noEmit": true,
+		"declaration": true,
+		"declarationDir": "dist",
+		"outDir": "dist",


logic: noEmit conflicts with declaration generation - declarations won't be written to disk

Suggested change

"noEmit": true,

"declaration": true,

"declarationDir": "dist",

"outDir": "dist",

"declaration": true,

"declarationDir": "dist",

"outDir": "dist",

Prompt To Fix With AI

This is a comment left during a code review. Path: templates/daemon/tsconfig.json.template Line: 9:12 Comment: **logic:** noEmit conflicts with declaration generation - declarations won't be written to disk ```suggestion "declaration": true, "declarationDir": "dist", "outDir": "dist", ``` How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2026-01-23T21:08:42Z

templates/daemon/tsconfig.json.template

+		"noImplicitAny": true,
+		"strictNullChecks": true,
+		"noUncheckedIndexedAccess": true


style: Missing strict checks: exactOptionalPropertyTypes, noPropertyAccessFromIndexSignature, and verbatimModuleSyntax from project standards

Suggested change

"noImplicitAny": true,

"strictNullChecks": true,

"noUncheckedIndexedAccess": true

"noImplicitAny": true,

"strictNullChecks": true,

"noUncheckedIndexedAccess": true,

"exactOptionalPropertyTypes": true,

"noPropertyAccessFromIndexSignature": true,

"verbatimModuleSyntax": true

Context Used: Context from dashboard - CLAUDE.md (source)

Prompt To Fix With AI

This is a comment left during a code review. Path: templates/daemon/tsconfig.json.template Line: 15:17 Comment: **style:** Missing strict checks: exactOptionalPropertyTypes, noPropertyAccessFromIndexSignature, and verbatimModuleSyntax from project standards ```suggestion "noImplicitAny": true, "strictNullChecks": true, "noUncheckedIndexedAccess": true, "exactOptionalPropertyTypes": true, "noPropertyAccessFromIndexSignature": true, "verbatimModuleSyntax": true ``` **Context Used:** Context from `dashboard` - CLAUDE.md ([source](https://app.greptile.com/review/custom-context?memory=72baf8a2-698b-40bf-966d-5b799d03ad3b)) How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2026-01-23T21:08:42Z

apps/outfitter/templates/daemon/tsconfig.json.template

+		"noEmit": true,
+		"declaration": true,
+		"declarationDir": "dist",
+		"outDir": "dist",


logic: noEmit: true conflicts with declaration: true - TypeScript cannot generate declaration files when emission is disabled

Suggested change

"noEmit": true,

"declaration": true,

"declarationDir": "dist",

"outDir": "dist",

"declaration": true,

"declarationDir": "dist",

"outDir": "dist",

Prompt To Fix With AI

This is a comment left during a code review. Path: apps/outfitter/templates/daemon/tsconfig.json.template Line: 9:12 Comment: **logic:** `noEmit: true` conflicts with `declaration: true` - TypeScript cannot generate declaration files when emission is disabled ```suggestion "declaration": true, "declarationDir": "dist", "outDir": "dist", ``` How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2026-01-23T21:08:43Z

apps/outfitter/templates/daemon/tsconfig.json.template

+		"noImplicitAny": true,
+		"strictNullChecks": true,
+		"noUncheckedIndexedAccess": true


style: Missing exactOptionalPropertyTypes, noPropertyAccessFromIndexSignature, and verbatimModuleSyntax options specified in CLAUDE.md

Suggested change

"noImplicitAny": true,

"strictNullChecks": true,

"noUncheckedIndexedAccess": true

"noImplicitAny": true,

"strictNullChecks": true,

"noUncheckedIndexedAccess":true,

"exactOptionalPropertyTypes": true,

"noPropertyAccessFromIndexSignature": true,

"verbatimModuleSyntax": true

Context Used: Context from dashboard - CLAUDE.md (source)

Prompt To Fix With AI

This is a comment left during a code review. Path: apps/outfitter/templates/daemon/tsconfig.json.template Line: 15:17 Comment: **style:** Missing `exactOptionalPropertyTypes`, `noPropertyAccessFromIndexSignature`, and `verbatimModuleSyntax` options specified in CLAUDE.md ```suggestion "noImplicitAny": true, "strictNullChecks": true, "noUncheckedIndexedAccess":true, "exactOptionalPropertyTypes": true, "noPropertyAccessFromIndexSignature": true, "verbatimModuleSyntax": true ``` **Context Used:** Context from `dashboard` - CLAUDE.md ([source](https://app.greptile.com/review/custom-context?memory=72baf8a2-698b-40bf-966d-5b799d03ad3b)) How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2026-01-23T21:08:44Z

apps/outfitter/templates/daemon/src/cli.ts.template

+				if (response.ok) {
+					logger.info`Daemon stopped`;
+				}


logic: Missing error handling when response.ok is false - daemon stop command may appear successful but actually fail

Suggested change

if (response.ok) {

logger.info`Daemon stopped`;

}

if (response.ok) {

logger.info`Daemon stopped`;

} else {

logger.error`Failed to stop daemon: ${response.status} ${response.statusText}`;

process.exit(1);

}

Prompt To Fix With AI

This is a comment left during a code review. Path: apps/outfitter/templates/daemon/src/cli.ts.template Line: 66:68 Comment: **logic:** Missing error handling when response.ok is false - daemon stop command may appear successful but actually fail ```suggestion if (response.ok) { logger.info`Daemon stopped`; } else { logger.error`Failed to stop daemon: ${response.status} ${response.statusText}`; process.exit(1); } ``` How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2026-01-23T21:08:44Z

apps/outfitter/templates/mcp/tsconfig.json.template

+		"noEmit": true,
+		"declaration": true,
+		"declarationDir": "dist",
+		"outDir": "dist",


logic: Configuration conflict: noEmit: true prevents TypeScript from generating output files, but declaration, declarationDir, and outDir are set for file generation

Suggested change

"noEmit": true,

"declaration": true,

"declarationDir": "dist",

"outDir": "dist",

"declaration": true,

"declarationDir": "dist",

"outDir": "dist",

Prompt To Fix With AI

This is a comment left during a code review. Path: apps/outfitter/templates/mcp/tsconfig.json.template Line: 9:12 Comment: **logic:** Configuration conflict: `noEmit: true` prevents TypeScript from generating output files, but `declaration`, `declarationDir`, and `outDir` are set for file generation ```suggestion "declaration": true, "declarationDir": "dist", "outDir": "dist", ``` How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2026-01-23T21:08:45Z

apps/outfitter/templates/mcp/tsconfig.json.template

+		"noImplicitAny": true,
+		"strictNullChecks": true,
+		"noUncheckedIndexedAccess": true


style: Missing strict TypeScript settings mentioned in CLAUDE.md: exactOptionalPropertyTypes, noPropertyAccessFromIndexSignature, and verbatimModuleSyntax

Suggested change

"noImplicitAny": true,

"strictNullChecks": true,

"noUncheckedIndexedAccess": true

"noImplicitAny": true,

"strictNullChecks": true,

"noUncheckedIndexedAccess": true,

"exactOptionalPropertyTypes": true,

"noPropertyAccessFromIndexSignature": true,

"verbatimModuleSyntax": true

Context Used: Context from dashboard - CLAUDE.md (source)

_{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}

Prompt To Fix With AI

This is a comment left during a code review. Path: apps/outfitter/templates/mcp/tsconfig.json.template Line: 15:17 Comment: **style:** Missing strict TypeScript settings mentioned in CLAUDE.md: `exactOptionalPropertyTypes`, `noPropertyAccessFromIndexSignature`, and `verbatimModuleSyntax` ```suggestion "noImplicitAny": true, "strictNullChecks": true, "noUncheckedIndexedAccess": true, "exactOptionalPropertyTypes": true, "noPropertyAccessFromIndexSignature": true, "verbatimModuleSyntax": true ``` **Context Used:** Context from `dashboard` - CLAUDE.md ([source](https://app.greptile.com/review/custom-context?memory=72baf8a2-698b-40bf-966d-5b799d03ad3b)) Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time! How can I resolve this? If you propose a fix, please make it concise.

galligan · 2026-01-23T23:20:46Z

Addressed greptile notes in outfitter templates: aligned CLI tsconfig (schema/lib/types/strict options; removed conflicting noEmit), removed extra console.log in CLI program template, and tightened MCP tool dispatch with handler map + arg guard. Restacked and resubmitted.

galligan mentioned this pull request Jan 23, 2026

feat(ui): add formatRelative() for human-readable timestamps #72

Open

galligan added the enhancement New feature or request label Jan 23, 2026

galligan marked this pull request as ready for review January 23, 2026 15:41

galligan force-pushed the p6/templates/mcp branch from e41a0ed to ffb2001 Compare January 23, 2026 16:03

galligan force-pushed the p6/templates/daemon branch from e0d862c to cb01622 Compare January 23, 2026 16:03

outfitter-dev deleted a comment from chatgpt-codex-connector bot Jan 23, 2026

This was referenced Jan 23, 2026

outfitter init does not replace {{projectName}}/{{binName}} placeholders #98

Open

outfitter init fails when installed (templates not packaged) #100

Open

galligan force-pushed the p6/templates/daemon branch from cb01622 to 0e991da Compare January 23, 2026 18:03

galligan force-pushed the p6/templates/mcp branch from ffb2001 to 08c120b Compare January 23, 2026 18:03

galligan force-pushed the p6/templates/daemon branch from 0e991da to f30bd92 Compare January 23, 2026 19:46

greptile-apps bot reviewed Jan 23, 2026

View reviewed changes

fix(templates): add dev:daemon script

e9e479f

galligan changed the base branch from p6/templates/mcp to graphite-base/94 January 23, 2026 23:18

fix(templates): align outfitter templates

ca74b08

galligan force-pushed the graphite-base/94 branch from 218d1dc to 9e371ae Compare January 23, 2026 23:20

galligan force-pushed the p6/templates/daemon branch from f30bd92 to ca74b08 Compare January 23, 2026 23:20

galligan changed the base branch from graphite-base/94 to p6/templates/mcp January 23, 2026 23:20

		logger.info`Hello, ${name}!`;
		console.log(`Hello, ${name}!`);

feat(templates): add daemon project template #94

Are you sure you want to change the base?

feat(templates): add daemon project template #94

Conversation

galligan commented Jan 23, 2026

Uh oh!

galligan commented Jan 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

greptile-apps bot commented Jan 23, 2026

Greptile Summary

Important Files Changed

Confidence score: 4/5

Sequence Diagram

Uh oh!

greptile-apps bot left a comment

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

galligan commented Jan 23, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

galligan commented Jan 23, 2026 •

edited

Loading