Skip to content

refactor(api,cli): split resume selector from bootstrap skipping#203

Merged
alexec merged 1 commit intokitproj:mainfrom
swedishborgie:split-resume-flag
Jan 28, 2026
Merged

refactor(api,cli): split resume selector from bootstrap skipping#203
alexec merged 1 commit intokitproj:mainfrom
swedishborgie:split-resume-flag

Conversation

@swedishborgie
Copy link
Collaborator

BREAKING CHANGE: The -r flag now only sets the resume=true selector and no longer skips rule discovery and bootstrap scripts. To achieve the previous behavior of skipping bootstrap, users must now use both -r and --skip-bootstrap flags together.

The change separates concerns:

  • -r: Sets resume selector for task filtering
  • --skip-bootstrap: Controls whether to discover rules, skills, and run bootstrap scripts

This allows users to independently control task filtering and bootstrap behavior, providing more flexibility in workflow configurations.

API changes:

  • Added WithBootstrap() option in options.go to control bootstrap behavior programmatically
  • WithResume() now only sets the resume selector and no longer affects bootstrap discovery
  • Context.doBootstrap field controls rule/skill discovery and bootstrap script execution independently from resume selector

BREAKING CHANGE: The -r flag now only sets the resume=true selector
and no longer skips rule discovery and bootstrap scripts. To achieve
the previous behavior of skipping bootstrap, users must now use both
-r and --skip-bootstrap flags together.

The change separates concerns:
- -r: Sets resume selector for task filtering
- --skip-bootstrap: Controls whether to discover rules, skills, and
  run bootstrap scripts

This allows users to independently control task filtering and bootstrap
behavior, providing more flexibility in workflow configurations.

API changes:
- Added WithBootstrap() option in options.go to control bootstrap
  behavior programmatically
- WithResume() now only sets the resume selector and no longer affects
  bootstrap discovery
- Context.doBootstrap field controls rule/skill discovery and bootstrap
  script execution independently from resume selector
{
name: "command output not trimmed",
content: "!`echo -n hello` world",
content: "!`printf 'hello'` world",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was to resolve a POSIX compatibility issue on macOS.

@alexec alexec merged commit 8617256 into kitproj:main Jan 28, 2026
1 check passed
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.

2 participants