feat: update Bob integration to skills-based layout for Bob 2.0 by davidebibm · Pull Request #3415 · github/spec-kit

davidebibm · 2026-07-08T18:26:37Z

Bob 2.0 replaces the command-based workflow with a skills-based layout.

https://bob.ibm.com/blog/bob-v2-release-announcement

Switch BobIntegration from MarkdownIntegration to SkillsIntegration
Update folder/dir from .bob/commands to .bob/skills
Change extension from .md to /SKILL.md (skills layout)
Add --skills option (default: True) consistent with Codex pattern
Update tests to inherit from SkillsIntegrationTests (28 tests pass)
Bump catalog entry to version 2.0.0 with updated description

Description

Testing

Tested locally with uv run specify --help
Ran existing tests with uv sync && uv run pytest
Tested with a sample project (if applicable)

AI Disclosure

I did not use AI assistance for this contribution
I did use AI assistance (describe below)

Very simple code generated by Bob but verified by me

Bob 2.0 replaces the command-based workflow (.bob/commands/*.md) with a skills-based layout (.bob/skills/speckit-<name>/SKILL.md), matching the pattern used by Claude Code, Codex, and other skills-first agents. - Switch BobIntegration from MarkdownIntegration to SkillsIntegration - Update folder/dir from .bob/commands to .bob/skills - Change extension from .md to /SKILL.md (skills layout) - Add --skills option (default: True) consistent with Codex pattern - Update tests to inherit from SkillsIntegrationTests (28 tests pass) - Bump catalog entry to version 2.0.0 with updated description Assisted-by: IBM Bob (model: claude-sonnet-4-5, autonomous)

davidebibm · 2026-07-09T12:20:14Z

@mnriem please review, we need to make this work with new Bob... Thankyou so much

Copilot

Pull request overview

This PR updates the built-in IBM Bob integration to align with Bob 2.0’s skills-based layout, switching installation output from command files to speckit-<name>/SKILL.md skills directories and bumping the integration’s catalog version accordingly.

Changes:

Migrate BobIntegration from MarkdownIntegration to SkillsIntegration and update output paths to .bob/skills/.../SKILL.md.
Update Bob integration tests to use the shared SkillsIntegrationTests mixin.
Bump the Bob entry in integrations/catalog.json to 2.0.0 with an updated description.

Show a summary per file

File	Description
`src/specify_cli/integrations/bob/__init__.py`	Switch Bob to `SkillsIntegration` and update registrar/config output to `.bob/skills` + `/SKILL.md`.
`tests/integrations/test_integration_bob.py`	Update base test mixin and expected output directories for the skills layout.
`integrations/catalog.json`	Bump Bob integration version/description to reflect the 2.0.0 skills-based update.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Files reviewed: 3/3 changed files
Comments generated: 2
Review effort level: Low

mnriem

As this fundamentally changes the layout for any Bob users what is the migration strategy? This will break them once they adopt a new version of Spec Kit so you need to make sure this goes through a deprecation cycle so they can migrate to the new Bob version. E.g make it an opt-in to the new version of Bob first and then in 2 minor releases (X.Y.Z where Y is minor) you can then remove the non-skill variant

davidebibm · 2026-07-10T16:02:29Z

As this fundamentally changes the layout for any Bob users what is the migration strategy? This will break them once they adopt a new version of Spec Kit so you need to make sure this goes through a deprecation cycle so they can migrate to the new Bob version. E.g make it an opt-in to the new version of Bob first and then in 2 minor releases (X.Y.Z where Y is minor) you can then remove the non-skill variant

Hi @mnriem
this should not break for Bob 1 user cause it was already compatible with skill mode.
Still needed this?

mnriem · 2026-07-13T12:18:20Z

Thanks @davidebibm — but I think we're talking about two different layers, and this still needs to change before it can land.

Your point is about Bob the tool — that Bob 1.x can already read the skills layout. My concern is about Spec Kit's generated output: this PR changes what specify writes for the bob integration from .bob/commands/*.md to .bob/skills/speckit-<name>/SKILL.md. Any project someone already initialized has .bob/commands/*.md on disk. When they upgrade Spec Kit and re-init or switch, that layout is orphaned/replaced. That's a breaking change in our contract regardless of whether the Bob app happens to read both directories.

A hard cutover isn't acceptable here — we shouldn't catch existing users off guard. This needs to go through a proper deprecation cycle:

This release: make skills an opt-in and keep the existing .bob/commands markdown layout as the default. That's real dual-mode — setup() branching between the two layouts, the way Copilot does it — not the --skills flag as written now.
Next cycle: flip the default to skills, with the legacy layout still available.
Cycle after that: remove the legacy markdown mode.

As it stands the --skills flag is a no-op: BobIntegration is hardwired to SkillsIntegration and the MarkdownIntegration variant is deleted, so --skills=False still emits skills. There's no path that produces the old layout, so it doesn't actually provide the opt-in it implies.

Can you rework it along those lines — genuine dual-mode with skills opt-in first — so we phase this in without breaking anyone?

davidebibm · 2026-07-13T12:28:51Z

Ok, thankyou @mnriem i'll do that

…s-integration

davidebibm · 2026-07-13T15:01:48Z

@mnriem Done, hope this is what you are expecting

Copilot

Review details

Files reviewed: 3/3 changed files
Comments generated: 3
Review effort level: Medium

+            IntegrationOption(
+                "--skills",
+                is_flag=True,
+                default=False,


+    def effective_invoke_separator(
+        self, parsed_options: dict[str, Any] | None = None
+    ) -> str:
+        """Return ``"-"`` when skills mode is requested, ``"."`` otherwise."""
+        if parsed_options and parsed_options.get("skills"):


+        helper = _BobSkillsHelper()
+        return SkillsIntegration.setup(helper, project_root, manifest, parsed_options, **opts)


mnriem · 2026-07-13T20:21:56Z

Please address Copilot feedback. You will need to update the description to reflect the reality

davidebibm requested a review from mnriem as a code owner July 8, 2026 18:26

mnriem requested a review from Copilot July 10, 2026 12:04

Copilot started reviewing on behalf of mnriem July 10, 2026 12:04 View session

Copilot AI reviewed Jul 10, 2026

View reviewed changes

Comment thread src/specify_cli/integrations/bob/__init__.py Outdated

Comment thread src/specify_cli/integrations/bob/__init__.py

mnriem requested changes Jul 10, 2026

View reviewed changes

davidebibm added 2 commits July 13, 2026 16:53

Merge remote-tracking branch 'origin/main' into feat/update-bob-skill…

e893245

…s-integration

PR comments fix: keep old Bob 1 commands till next release

dea23f0

davidebibm requested a review from mnriem July 13, 2026 15:01

mnriem requested a review from Copilot July 13, 2026 19:23

Copilot started reviewing on behalf of mnriem July 13, 2026 19:24 View session

Copilot AI reviewed Jul 13, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: update Bob integration to skills-based layout for Bob 2.0#3415

feat: update Bob integration to skills-based layout for Bob 2.0#3415
davidebibm wants to merge 3 commits into
github:mainfrom
davidebibm:feat/update-bob-skills-integration

davidebibm commented Jul 8, 2026 •

edited

Loading

Uh oh!

davidebibm commented Jul 9, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

mnriem left a comment

Uh oh!

davidebibm commented Jul 10, 2026

Uh oh!

mnriem commented Jul 13, 2026

Uh oh!

davidebibm commented Jul 13, 2026

Uh oh!

davidebibm commented Jul 13, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

mnriem commented Jul 13, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		helper = _BobSkillsHelper()
		return SkillsIntegration.setup(helper, project_root, manifest, parsed_options, **opts)

Uh oh!

Conversation

davidebibm commented Jul 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Testing

AI Disclosure

Uh oh!

davidebibm commented Jul 9, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Review details

Uh oh!

Uh oh!

Uh oh!

mnriem left a comment

Choose a reason for hiding this comment

Uh oh!

davidebibm commented Jul 10, 2026

Uh oh!

mnriem commented Jul 13, 2026

Uh oh!

davidebibm commented Jul 13, 2026

Uh oh!

davidebibm commented Jul 13, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Review details

Uh oh!

mnriem commented Jul 13, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

davidebibm commented Jul 8, 2026 •

edited

Loading