Conversation
…p test gates - BUILT_IN_WORKFLOWS gains library (plan, build, review) and library-secure (plan, build, security, review) — review runs the suite, the correct verification when there is no live app. - resolveWorkflow honors a new default_workflow config key when resolving the name 'default': flag > ticket frontmatter > user workflows.default > default_workflow > built-in. Unknown names get a .bobbyrc.yml hint. - deriveDefaultWorkflow(stack, config): explicit stack field wins; else library iff no health checks and no dev command. Pure, called at init/new, never inside scaffoldProject (BOB-123). - Every testing-stage prompt path (orchestration, sprint, feature, slow mode, single/batch test runs) takes a default-true hasHealthChecks and, when false, instructs blocking the ticket with 'no live app to test — use the library workflow' instead of launching bobby-test. Wording lives in one helper; the reason string appears exactly once in source. TKT-049 invariant block and target-matrix suite green with zero edits. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
An overridden built-in printed the built-in stage list next to '(overridden)' — now the row shows the override's resolved steps. When default_workflow is set, list names it as the project default. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…a comment - writeConfigCommented emits default_workflow near the top with an explanatory comment (why, and how to undo); the workflows hint block now names library and library-secure. - bobby init and bobby new call deriveDefaultWorkflow at their config construction sites — outside scaffoldProject, so refresh stays byte-identical (BOB-123). - bobby sprint threads hasHealthChecks into buildSprintPrompt. - bobby run help text names library among the built-ins. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…plate, README - bobby-test skill (template + tracked scaffold copy): runtime-phrased no-live-app guard — block the ticket with the library-workflow reason instead of emitting BLOCKED test cases one by one. - CLAUDE.md template's built-in workflow list names library/library-secure. - README documents both workflows, when Bobby picks library automatically, how to opt out, and the default-name shadowing caveat. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…heck The no-live-app gate now replaces the testing-stage launch with a block prompt whenever config has no health_checks, and the FSM harness's fake executor parsed 'block' as a stage. Every scenario in this file is a live-app project running a test-ending workflow, so the fixture carries a health check unless a test brings its own key. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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
1 participant
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.
Stacked on
integrate/app-studio(merge #14 first).library[plan→build→review] andlibrary-secure[plan→build→security→review] added toBUILT_IN_WORKFLOWS.default_workflowconfig key, derived stack-aware atbobby init/createProject(explicit stack field wins; elselibraryiff no health checks and no dev command), written to.bobbyrc.ymlwith an explanatory comment. Resolver precedence: flag > ticket frontmatter > userworkflows.default>default_workflow> built-in; unknown values throw with a.bobbyrc.ymlhint.workflow listshows effective stages and the project default.Review approved (zero blocking findings, all six ACs). Live-tested against scratch library/CLI and web projects: a library ticket runs backlog→shipping with no testing stage; all three run paths produce the block instruction when forced into testing. Full suite 1458 passed / 0 failed; lint 0 errors; target-matrix and workflow invariant suites green with zero per-target edits.
🤖 Generated with Claude Code