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

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

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

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

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

davidebibm · 2026-07-14T06:53:03Z

@mnriem Addressed Copilot comments and aligned with main

Copilot

Review details

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

mnriem · 2026-07-14T14:30:35Z

Please address Copilot feedback and resolve conflicts

…te legacy commands to opt-in

…to ALWAYS_SLASH_AGENTS - init.py: suppress ai_skills=True when --legacy-commands is passed so extensions and presets target .bob/commands, not .bob/skills - _invocation_style.py: add 'bob' to ALWAYS_SLASH_AGENTS so init next-steps and hook invocations always show /speckit-<name> (skills is the default layout; no ai_skills flag required)

davidebibm · 2026-07-14T14:52:08Z

@mnriem Solved Copilot fb and rebased to avoid get Copilot feedback on other devs code 👍

Copilot

Review details

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

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

…killsIntegration) - bob/__init__.py: switch BobIntegration base from SkillsIntegration to IntegrationBase; add _BobSkillsHelper for skills-mode delegation; set invoke_separator='-' explicitly; set _skills_mode flag in setup() so consumers can derive the effective mode without isinstance checks - _helpers.py: replace isinstance(integration, SkillsIntegration) guard with getattr(_skills_mode) so legacy-commands mode does not persist ai_skills=True - _invocation_style.py: remove 'bob' from ALWAYS_SLASH_AGENTS — Bob 2.0 skills are invoked via natural language, not /skill-name slash commands - integrations/catalog.json: advance updated_at to 2026-07-15

davidebibm · 2026-07-14T15:23:22Z

@mnriem Solved Copilot fb again

Copilot

Review details

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

+        self._skills_mode = True
+        return SkillsIntegration.setup(
+            _BobSkillsHelper(), project_root, manifest, parsed_options, **opts
+        )


+    registrar_config = {
+        "dir": ".bob/skills",
+        "format": "markdown",
+        "args": "$ARGUMENTS",
+        "extension": "/SKILL.md",


Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

davidebibm · 2026-07-14T16:08:36Z

@mnriem Are we sure there will be a Copilor PR without comments at some point? :-)

Copilot

Review details

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

    if script_type:
        opts["script"] = script_type
-    if isinstance(integration, SkillsIntegration) or getattr(integration, "_skills_mode", False):
+    if getattr(integration, "_skills_mode", False):


+    # Set by setup() to reflect the active mode; read by _helpers.py and
+    # init.py via getattr(integration, "_skills_mode", False).
+    _skills_mode: bool = False


+    key = "bob"
+    invoke_separator = "-"


+    registrar_config = {
+        "dir": ".bob/skills",
+        "format": "markdown",
+        "args": "$ARGUMENTS",
+        "extension": "/SKILL.md",


mnriem · 2026-07-14T16:46:38Z

@davidebibm Yes, eventually. Reviews tend to take a bit longer because it wants all those corner cases addressed, but it does not find them all in one go. Please address the current Copilot feedback

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

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

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

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

Copilot suggested change

1c3d1cf

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

mnriem requested a review from Copilot July 14, 2026 12:49

Copilot started reviewing on behalf of mnriem July 14, 2026 12:49 View session

Copilot AI reviewed Jul 14, 2026

View reviewed changes

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

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

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

Comment thread CHANGELOG.md Outdated

feat(bob): address copilot comments, make skills layout default, demo…

674a8db

…te legacy commands to opt-in

davidebibm force-pushed the feat/update-bob-skills-integration branch from 6b3a18c to 674a8db Compare July 14, 2026 14:44

mnriem requested a review from Copilot July 14, 2026 14:58

Copilot started reviewing on behalf of mnriem July 14, 2026 14:58 View session

Copilot AI reviewed Jul 14, 2026

View reviewed changes

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

Comment thread src/specify_cli/_invocation_style.py Outdated

Comment thread integrations/catalog.json Outdated

Comment thread integrations/catalog.json

davidebibm and others added 2 commits July 14, 2026 17:14

Copilot suggestion

d5e0e95

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

fix(lint): remove unused SkillsIntegration import from _helpers.py

64c0b9d

mnriem requested a review from Copilot July 14, 2026 15:28

Copilot started reviewing on behalf of mnriem July 14, 2026 15:28 View session

Copilot AI reviewed Jul 14, 2026

View reviewed changes

Copilot suggested change

6ab8d61

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

mnriem requested a review from Copilot July 14, 2026 16:20

Copilot started reviewing on behalf of mnriem July 14, 2026 16:21 View session

Copilot AI reviewed Jul 14, 2026

View reviewed changes

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!

Uh oh!

Uh oh!

Uh oh!

mnriem commented Jul 13, 2026

Uh oh!

davidebibm commented Jul 14, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Review details

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mnriem commented Jul 14, 2026

Uh oh!

davidebibm commented Jul 14, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Review details

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

davidebibm commented Jul 14, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Review details

Uh oh!

Uh oh!

Uh oh!

davidebibm commented Jul 14, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Review details

Uh oh!

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

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

davidebibm commented Jul 8, 2026 •

edited

Loading

mnriem commented Jul 14, 2026 •

edited

Loading