Skip to content

Comments

proposal: Consolidate optional local dev services into monorepo#2058

Draft
nick-inkeep wants to merge 2 commits intomainfrom
proposal/consolidate-optional-services
Draft

proposal: Consolidate optional local dev services into monorepo#2058
nick-inkeep wants to merge 2 commits intomainfrom
proposal/consolidate-optional-services

Conversation

@nick-inkeep
Copy link
Collaborator

@nick-inkeep nick-inkeep commented Feb 17, 2026

Summary

Proposal spec for consolidating the agents-optional-local-dev companion repo directly into the agents monorepo.

Spec: docs/proposals/consolidate-optional-services.md

Problem

Optional local dev services (Nango, SigNoz, OTEL Collector, Jaeger) live in a separate companion repo. A bootstrap shim clones it on demand. This creates coordination overhead (paired PRs, cross-repo sync) for ~3.3MB of configs with no legal barrier to inlining.

Proposed change

  1. Move companion repo contents into optional-services/ at monorepo root
  2. Rewrite setup-optional.sh — absorb companion's setup.sh directly, eliminating the clone/delegate shim
  3. Auto-sync to create-agents-template/ via lint-staged + rsync
  4. Update all docs (3 local dev + 5 deployment guides + contributing)
  5. Archive companion repo with redirect README

What stays the same

  • All 4 pnpm commands: setup-dev:optional, optional:stop, optional:status, optional:reset
  • All 9 env vars written to .env
  • Docker service behavior
  • Core services (docker-compose.dbs.yml)

Pending decisions for implementer

  • D3: rsync in lint-staged — fires per-file (lint-staged limitation) but idempotent; alternative is cp -R in a pre-commit hook
  • D4: Drop OPTIONAL_SERVICES_DIR env var override — recommend dropping, trivial to re-add

Implementation notes

  • biome.jsonc has ignoreUnknown: false — committed YAML/XML in optional-services/ will need a Biome exclusion
  • Recommend adding explicit name: optional-services to the compose file for stable Docker project naming
  • Surface area inventory in the spec covers 21 items across 7 categories with line numbers

🤖 Generated with Claude Code

@changeset-bot
Copy link

changeset-bot bot commented Feb 17, 2026

⚠️ No Changeset found

Latest commit: 1c99906

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Feb 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agents-api Ready Ready Preview, Comment Feb 19, 2026 2:33am
agents-docs Ready Ready Preview, Comment Feb 19, 2026 2:33am
agents-manage-ui Ready Ready Preview, Comment Feb 19, 2026 2:33am

Request Review

nick-inkeep and others added 2 commits February 18, 2026 18:29
Add SPEC.md for eliminating the companion repo (agents-optional-local-dev) by
moving all Docker Compose configs, service configs, and setup automation
directly into the agents monorepo and create-agents-template.

Key points:
- Move configs to optional-services/ in monorepo root
- Rewrite setup-optional.sh to contain full logic (no shim/clone)
- Auto-sync to create-agents-template via lint-staged + rsync
- Update all docs (3 local dev + 5 deployment guides)
- Archive companion repo with redirect README
- User-facing pnpm commands unchanged

See the spec for full surface area inventory, technical design, migration
path, and acceptance criteria.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Remove all references to internal PR evolution (#2052, #8), resolved
open questions, and verbose explanations. Frame everything as current
state → target state.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant