chore(outfitter): sync templates on pack

[galligan]

Summary

• Sync templates during pack to keep published artifacts up to date.
• Add script wiring for template sync.

Changes

• apps/outfitter/package.json
• scripts/sync-templates.ts

Testing

• Not run (not requested).

[galligan]

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

• chore(outfitter): sync templates on pack #108 👈 (View in Graphite)
• chore(changesets): ignore app package #107
• fix(errors): avoid TaggedError extends warnings #106
• chore(release): align package versions to 0.1.0 #105
• docs: add README and CHANGELOG for testing, kit, outfitter #95
• feat(templates): add daemon project template #94
• feat(templates): add MCP server project template #93
• feat(templates): add CLI project template #92
• fix(cli): implement confirmDestructive prompts #104
• fix(cli): implement createCLI and command builder #103
• feat(kit): add @outfitter/kit meta-package #91
• docs(daemon): add README and CHANGELOG #90
• fix(daemon): fix locking module bugs (GREEN phase) #89
• test(daemon): add locking module tests (RED phase) #88
• feat(daemon): add errors, platform, and locking module stubs #87
• feat(outfitter): add doctor command for environment validation #86
• feat(outfitter): add init command with project scaffolding #85
• feat(daemon): add IPC communication and health checks #84
• feat(daemon): add types and lifecycle management #83
• feat(index): add FTS5 full-text search implementation #82
• feat(index): add types for FTS5 full-text search #81
• feat(testing): add MCP and CLI test harnesses #80
• feat(testing): add test fixture utilities #79
• feat(mcp): add MCP server implementation with typed handlers #78
• feat(mcp): add types and interfaces for MCP server #77
• feat(ui): add Shape discriminated union and render() #76
• feat(ui): add createTokens semantic color layer #75
• feat(state): add pagination helpers #74
• feat(logging): add DEFAULT_PATTERNS for common secret redaction #73
• feat(ui): add formatRelative() for human-readable timestamps #72
• feat(state): add encodeCursor/decodeCursor for pagination #71
• feat(types): add hashId() for deterministic short identifiers #70
• feat(contracts): add Logger.child() and fix AssertionError category #69
• feat(config): centralize environment variable access with Zod validation #42
• fix(logging): enable redaction by default when global rules exist #41
• docs(ui): add README, CHANGELOG, and TSDoc comments #39
• docs(state): add README, CHANGELOG, and TSDoc comments #38
• docs(file-ops): add README, CHANGELOG, and TSDoc comments #37
• docs(logging): add README, CHANGELOG, and TSDoc comments #36
• docs(config): add README, CHANGELOG, and TSDoc comments #35
• fix(cli): enforce cursor TTL #114
• fix(tests): remove placeholder tests #113
• chore(gitattributes): prefer ours for bun lockfiles #115
• main

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

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

[greptile-apps]

Greptile Summary

• Adds template synchronization automation to ensure the outfitter CLI package includes up-to-date templates when published
• Introduces a new sync script (scripts/sync-templates.ts) and package lifecycle hooks (prepack) to automatically copy templates from root to app directory
• Establishes single source of truth for templates in root /templates/ while maintaining app-local copies for publishing

Important Files Changed

Filename	Overview
scripts/sync-templates.ts	New script that copies templates from root templates/ to apps/outfitter/templates/ using recursive file operations
apps/outfitter/package.json	Added sync:templates script and prepack lifecycle hook to automate template synchronization before packaging

Confidence score: 4/5

• This PR is safe to merge with minimal risk of breaking functionality
• Score reflects straightforward file operations with proper error handling, though the script lacks explicit error handling for individual operations
• Pay attention to the sync script's file operations to ensure proper permissions and cross-platform compatibility

Sequence Diagram

sequenceDiagram
    participant User
    participant NPM as "NPM/Bun Pack"
    participant Package as "apps/outfitter/package.json"
    participant Script as "scripts/sync-templates.ts"
    participant FileSystem as "File System"

    User->>NPM: "bun pack"
    NPM->>Package: "Execute prepack hook"
    Package->>Script: "Run sync:templates script"
    Script->>FileSystem: "Remove destination directory"
    Script->>FileSystem: "Create destination directory"
    Script->>FileSystem: "Copy templates recursively"
    Script-->>Package: "Templates synced"
    Package-->>NPM: "Prepack complete"
    NPM-->>User: "Package created with synced templates"

Loading

[greptile-apps]

_{2 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

logic: The operations run sequentially without error handling - if any step fails, the script will crash and leave the destination in an inconsistent state

Prompt To Fix With AI

This is a comment left during a code review.
Path: scripts/sync-templates.ts
Line: 10:12

Comment:
**logic:** The operations run sequentially without error handling - if any step fails, the script will crash and leave the destination in an inconsistent state

How can I resolve this? If you propose a fix, please make it concise.

[galligan]

Added basic error handling to template sync script (try/catch with non-zero exit code) to make failures explicit. Restacked and resubmitted.

[galligan]

Follow-up: swapped console usage for stderr write + error string to satisfy lint, then resubmitted.