refactor!: rename to @codemcp/workflows with ade-workflows binary#229
Merged
Conversation
Remove two categories of self-transitions: - Abandon self-transitions on initial phases (greenfield, minor, posts, slides, tdd): semantically nonsensical since you can't abandon work you haven't started; reset_development() is the correct tool for abandonment - Terminal "done" self-transitions (business-analysis, c4-analysis, sdd-bugfix, sdd-feature, sdd-greenfield): redundant with default_instructions and misleading since they imply looping back when the workflow is actually complete https://claude.ai/code/session_01UwFmXVqmJePHvH3rEXPU4H
- Replace 'mark done with [x]' assertions with 'tasks in other tools' and 'Maintain `' to match the updated workflow continuity text - Remove gitBranch assertion from contract test as branch is not embedded in instruction output (only used for variable substitution) - Fix sdd-greenfield.yaml document state with empty transitions list to resolve state machine loader validation error
…nary Harmonize naming across the @codemcp/* ecosystem so developers have a consistent, predictable experience. Previously, package names and binary names varied wildly (responsible-vibe-mcp, agentic-knowledge, skills, mcp-prompts-server, etc.), forcing users to look up documentation for each tool. - Root package renamed from `responsible-vibe-mcp` to `@codemcp/workflows` - Binary renamed from `responsible-vibe-mcp` to `ade-workflows` - MCP server sub-package renamed to `@codemcp/workflows-server` (internal) - All documentation updated to use `npx @codemcp/workflows` invocation - The routing behavior is unchanged: no args starts MCP server, with args runs CLI - Breaking change for users with MCP configs using the old binary name - Users invoking via `npx @codemcp/workflows` are unaffected (npm resolves the binary) - Stub package `responsible-vibe-mcp` should be published pointing to the new name
BREAKING CHANGE: Package renamed from responsible-vibe-mcp to @codemcp/workflows - Rename package to @codemcp/workflows (CLI) and @codemcp/workflows-server (MCP) - Binary names: ade-workflows and ade-workflows-server - Use tsup bundler for self-contained packages - Mark internal packages as private - Update pnpm to 10.32.1
mrsimpson
force-pushed
the
ade-consistent-naming
branch
from
0a48db3 to
2afc404
Compare
The tsup bundling doesn't include resource files (YAML workflows, templates). Add copy-resources script to copy resources from core to mcp-server during build. This fixes the e2e test failure where workflows couldn't be found when the bundled server runs.
mrsimpson
force-pushed
the
ade-consistent-naming
branch
from
f5f9127 to
b79c875
Compare
Turbo was only caching dist/** but not resources/**, so the mcp-server resources weren't being restored from cache in CI.
Instead of bundling core (which loses resource files), keep it as an external runtime dependency like with tsc. This way Node.js resolves resources from node_modules/@codemcp/workflows-core/resources/. Reverts the copy-resources workaround which was error-prone.
Since @codemcp/workflows-core is private (not published), it must be bundled into mcp-server. But tsup only bundles JS, not resource files. Solution: - Bundle core with noExternal (for JS code) - Copy resources from core to mcp-server during build - Add resources/** to turbo outputs so cache includes them - Add resources to .gitignore (build artifact)
The e2e tests spawn the server via packages/cli/dist/index.js, which also bundles @codemcp/workflows-core. The bundled code looks for resources at ../resources/ relative to itself, so CLI needs its own copy of resources.
…onsible-vibe-mcp' - Updated agent YAML files to use @codemcp/workflows-server@latest - Updated skill-generator.test.ts expectations for 'workflows' key - Updated config-generator.test.ts expectations for 'workflows' key - Updated cli-agents.test.ts to expect new package name
The workflow resource handler was navigating 4 levels up from the bundled dist/index.js, which went outside the package directory. Since tsup bundles everything into dist/index.js, we only need to go 1 level up from dist/ to reach the package root where resources/ lives.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
@codemcp/workflows(CLI) and@codemcp/workflows-server(MCP)ade-workflowsandade-workflows-serverBreaking Change
Package renamed from
responsible-vibe-mcpto@codemcp/workflows. Users should update their MCP configuration:{ "mcpServers": { "workflows": { "command": "npx", "args": ["@codemcp/workflows-server"] } } }Part of ADE Naming Convention
This is part of harmonizing naming across all
@codemcp/*packages for the Agentic Development Environment.