From 6251353fc6f4402c486dba0758d81a65f13deed2 Mon Sep 17 00:00:00 2001 From: dbpolito Date: Tue, 24 Mar 2026 23:42:46 -0300 Subject: [PATCH 1/2] fix: fallback to existing bundled skills root - prefer a bundled skills root that contains skill directories while still falling back to an existing path - stop enabling the draft kompass-workflows skill in base config and generated artifacts - update bundled skill docs and readmes to reflect the current draft-only status --- kompass.jsonc | 6 ------ packages/core/kompass.jsonc | 6 ------ packages/core/skills/README.md | 4 ++-- packages/opencode/.opencode/kompass.jsonc | 7 ------- packages/opencode/.opencode/skills/README.md | 4 ++-- packages/opencode/config.ts | 21 +++++++++++++++++-- packages/opencode/kompass.jsonc | 6 ------ .../docs/docs/reference/skills/index.mdx | 4 +++- .../reference/skills/kompass-workflows.mdx | 7 ++++--- 9 files changed, 30 insertions(+), 35 deletions(-) diff --git a/kompass.jsonc b/kompass.jsonc index 3524381..ed33dbe 100644 --- a/kompass.jsonc +++ b/kompass.jsonc @@ -54,12 +54,6 @@ "summarize-changes": { "enabled": true }, }, - "skills": { - "entries": { - "kompass-workflows": { "enabled": true }, - }, - }, - "defaults": { "baseBranch": "main", }, diff --git a/packages/core/kompass.jsonc b/packages/core/kompass.jsonc index 3524381..ed33dbe 100644 --- a/packages/core/kompass.jsonc +++ b/packages/core/kompass.jsonc @@ -54,12 +54,6 @@ "summarize-changes": { "enabled": true }, }, - "skills": { - "entries": { - "kompass-workflows": { "enabled": true }, - }, - }, - "defaults": { "baseBranch": "main", }, diff --git a/packages/core/skills/README.md b/packages/core/skills/README.md index 3a51c89..9403124 100644 --- a/packages/core/skills/README.md +++ b/packages/core/skills/README.md @@ -1,2 +1,2 @@ -This directory intentionally exists so packaged OpenCode builds always ship a -concrete `skills/` path, even when no bundled skill files are present yet. +This directory contains bundled Kompass skills that ship with packaged OpenCode +builds. Keep a concrete `skills/` path available for runtime registration. diff --git a/packages/opencode/.opencode/kompass.jsonc b/packages/opencode/.opencode/kompass.jsonc index 662e365..aa444d0 100644 --- a/packages/opencode/.opencode/kompass.jsonc +++ b/packages/opencode/.opencode/kompass.jsonc @@ -96,13 +96,6 @@ "enabled": true } }, - "skills": { - "entries": { - "kompass-workflows": { - "enabled": true - } - } - }, "defaults": { "baseBranch": "main" }, diff --git a/packages/opencode/.opencode/skills/README.md b/packages/opencode/.opencode/skills/README.md index 3a51c89..9403124 100644 --- a/packages/opencode/.opencode/skills/README.md +++ b/packages/opencode/.opencode/skills/README.md @@ -1,2 +1,2 @@ -This directory intentionally exists so packaged OpenCode builds always ship a -concrete `skills/` path, even when no bundled skill files are present yet. +This directory contains bundled Kompass skills that ship with packaged OpenCode +builds. Keep a concrete `skills/` path available for runtime registration. diff --git a/packages/opencode/config.ts b/packages/opencode/config.ts index 5ca9ab8..f00ff9c 100644 --- a/packages/opencode/config.ts +++ b/packages/opencode/config.ts @@ -39,12 +39,29 @@ async function pathExists(filePath: string): Promise { } } +async function hasBundledSkillDirectories(root: string): Promise { + try { + const entries = await readdir(root, { withFileTypes: true }); + return entries.some((entry) => entry.isDirectory()); + } catch { + return false; + } +} + async function resolveBundledSkillsRoot(): Promise { + let firstExistingCandidate: string | undefined; + for (const candidate of BUNDLED_SKILL_ROOT_CANDIDATES) { - if (await pathExists(candidate)) return candidate; + if (!await pathExists(candidate)) continue; + + firstExistingCandidate ??= candidate; + + if (await hasBundledSkillDirectories(candidate)) { + return candidate; + } } - return undefined; + return firstExistingCandidate; } type ApplyConfigOptions = { diff --git a/packages/opencode/kompass.jsonc b/packages/opencode/kompass.jsonc index 3524381..ed33dbe 100644 --- a/packages/opencode/kompass.jsonc +++ b/packages/opencode/kompass.jsonc @@ -54,12 +54,6 @@ "summarize-changes": { "enabled": true }, }, - "skills": { - "entries": { - "kompass-workflows": { "enabled": true }, - }, - }, - "defaults": { "baseBranch": "main", }, diff --git a/packages/web/src/content/docs/docs/reference/skills/index.mdx b/packages/web/src/content/docs/docs/reference/skills/index.mdx index 9d3a6d7..a912b6b 100644 --- a/packages/web/src/content/docs/docs/reference/skills/index.mdx +++ b/packages/web/src/content/docs/docs/reference/skills/index.mdx @@ -5,12 +5,14 @@ description: Overview of skill support in Kompass. Kompass supports bundled and plugin-provided skills through config-driven filtering. -The base config currently enables the `kompass-workflows` skill entry. +The base config does not enable any bundled Kompass skills yet. ## Bundled skills ### `kompass-workflows` +Draft only for now; not bundled or enabled by default. + Bundled Kompass workflow skill entry enabled by the base config. - enabled through `skills.entries` diff --git a/packages/web/src/content/docs/docs/reference/skills/kompass-workflows.mdx b/packages/web/src/content/docs/docs/reference/skills/kompass-workflows.mdx index d2239db..5fe6376 100644 --- a/packages/web/src/content/docs/docs/reference/skills/kompass-workflows.mdx +++ b/packages/web/src/content/docs/docs/reference/skills/kompass-workflows.mdx @@ -1,13 +1,14 @@ --- title: kompass-workflows -description: Bundled Kompass workflow skill entry enabled by the base config. +description: Draft workflow skill content that is not bundled by default yet. --- ## Current status -The base config enables `kompass-workflows` through `skills.entries`. +`kompass-workflows` is still draft content and is not enabled in the base config. ## Notes -- bundled Kompass skills are registered automatically by the OpenCode adapter +- bundled Kompass skill paths are registered automatically by the OpenCode adapter - the `packages/core/skills/` directory is kept present so builds always ship a concrete skills path +- draft skill content currently lives outside the bundled skills directory until it is ready to ship From d29900e0323cd497eecfbf946d192908d501e0b1 Mon Sep 17 00:00:00 2001 From: dbpolito Date: Tue, 24 Mar 2026 23:50:19 -0300 Subject: [PATCH 2/2] fix: remove stale workflow skill docs --- .../content/docs/docs/reference/skills/index.mdx | 10 +--------- .../docs/reference/skills/kompass-workflows.mdx | 14 -------------- 2 files changed, 1 insertion(+), 23 deletions(-) delete mode 100644 packages/web/src/content/docs/docs/reference/skills/kompass-workflows.mdx diff --git a/packages/web/src/content/docs/docs/reference/skills/index.mdx b/packages/web/src/content/docs/docs/reference/skills/index.mdx index a912b6b..b2e08f3 100644 --- a/packages/web/src/content/docs/docs/reference/skills/index.mdx +++ b/packages/web/src/content/docs/docs/reference/skills/index.mdx @@ -9,12 +9,4 @@ The base config does not enable any bundled Kompass skills yet. ## Bundled skills -### `kompass-workflows` - -Draft only for now; not bundled or enabled by default. - -Bundled Kompass workflow skill entry enabled by the base config. - -- enabled through `skills.entries` -- registered automatically by the OpenCode adapter -- backed by a concrete bundled `skills/` path even when the directory is intentionally sparse +None yet. diff --git a/packages/web/src/content/docs/docs/reference/skills/kompass-workflows.mdx b/packages/web/src/content/docs/docs/reference/skills/kompass-workflows.mdx deleted file mode 100644 index 5fe6376..0000000 --- a/packages/web/src/content/docs/docs/reference/skills/kompass-workflows.mdx +++ /dev/null @@ -1,14 +0,0 @@ ---- -title: kompass-workflows -description: Draft workflow skill content that is not bundled by default yet. ---- - -## Current status - -`kompass-workflows` is still draft content and is not enabled in the base config. - -## Notes - -- bundled Kompass skill paths are registered automatically by the OpenCode adapter -- the `packages/core/skills/` directory is kept present so builds always ship a concrete skills path -- draft skill content currently lives outside the bundled skills directory until it is ready to ship