You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Commit 4884136
Browse filesBrowse the repository at this point in the historyBrowse files
Copy file name to clipboardExpand all lines: aidd_docs/tasks/2026_09/2026_09_29_mistral-support/brainstorm.md
+4-4Lines changed: 4 additions & 4 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -23,7 +23,7 @@ Reconstructed from the 13 commits between `main` and the archive branch `vibe/mi
23
23
24
24
### 2. Hook friction, Sep 4
25
25
26
-
The pre-commit suite rejected the worktree, not the feature. Root causes in [2026_09_04_precommit-hooks-debug.md](../2026_09_04_precommit-hooks-debug.md): the argument-hint parser required `---\n` while the CRLF worktree had `---\r\n`; the markdown-links walker scanned `.aidd/` (1626 cached files) because `SKIPPED_DIRS` omitted it; Biome tripped on 332 worktree files whose CRLF diverged from their LF blobs. One commit fixed the checkers. The next fixed the same `.`-does-not-match-`\r` class in the catalog summarizer, whose `parseFrontmatter` had silently dropped every SKILL.md `description`, and regenerated the seven `plugins/*/CATALOG.md` files.
26
+
The pre-commit suite rejected the worktree, not the feature. Root causes in [2026_09_04_precommit-hooks-debug.md](./debug/2026_09_04_precommit-hooks-debug.md): the argument-hint parser required `---\n` while the CRLF worktree had `---\r\n`; the markdown-links walker scanned `.aidd/` (1626 cached files) because `SKIPPED_DIRS` omitted it; Biome tripped on 332 worktree files whose CRLF diverged from their LF blobs. One commit fixed the checkers. The next fixed the same `.`-does-not-match-`\r` class in the catalog summarizer, whose `parseFrontmatter` had silently dropped every SKILL.md `description`, and regenerated the seven `plugins/*/CATALOG.md` files.
27
27
28
28
### 3. The feature iterated, Sep 4
29
29
@@ -35,12 +35,12 @@ One commit closed a registry gap: a plugins-capable tool with no `MARKETPLACE_PR
35
35
36
36
### 5. Truthful hook skip, Sep 8
37
37
38
-
Flat builds skip `hooks/` because the mistral flat contract has no `HasHooks`. The investigation in [2026_09_08_mistral-flat-hooks-skip-debug.md](../2026_09_08_mistral-flat-hooks-skip-debug.md) confirmed the skip is format-correct: Vibe loads `.vibe/hooks.toml` (`pre_tool`/`post_tool`/`post_agent`), never Claude's `hooks.json`, and `SessionStart` has no equivalent. The generic and OpenCode skip reasons were wrong for Mistral; the contract now carries a Mistral-specific `skipReason`. The marketplace contract still copies Claude `hooks.json`, which Vibe never loads.
38
+
Flat builds skip `hooks/` because the mistral flat contract has no `HasHooks`. The investigation in [2026_09_08_mistral-flat-hooks-skip-debug.md](./debug/2026_09_08_mistral-flat-hooks-skip-debug.md) confirmed the skip is format-correct: Vibe loads `.vibe/hooks.toml` (`pre_tool`/`post_tool`/`post_agent`), never Claude's `hooks.json`, and `SessionStart` has no equivalent. The generic and OpenCode skip reasons were wrong for Mistral; the contract now carries a Mistral-specific `skipReason`. The marketplace contract still copies Claude `hooks.json`, which Vibe never loads.
39
39
40
40
### 6. Flat build on a real Vibe install, investigated Sep 8, committed Sep 29
41
41
42
-
- EISDIR, root cause and fix in [2026_09_08_eisdir-banner-txt-debug.md](../2026_09_08_eisdir-banner-txt-debug.md): the Sep 3 mapper's `<path>/skill.md` wrap left directories on the new dest paths (an `assets/banner.txt/` directory holding the banner); `--force` then wrote through the directory and hit `EISDIR`. The fix: `checkCollision` deletes a leftover dest directory when `--force` is set; flat skills move to the generic `<plugin>-<skill>/SKILL.md` layout; a shared `mistral-skill-frontmatter` module emits `name`, `description`, `user-invocable` for both the build and the install paths.
43
-
- Vibe rejected the built SKILL.md, root cause and fix in [2026_09_08_vibe-skill-description-debug.md](../2026_09_08_vibe-skill-description-debug.md): `parseYamlLike` dropped every CRLF key, so the built file carried only `user-invocable` and a rewritten `name`, while Vibe requires `description`. The fix: `parseFrontmatter` splits on `/\r?\n/`, the same class of fix as the catalog summarizer's. Rebuild verified 47/47 SKILL.md carrying `description`.
42
+
- EISDIR, root cause and fix in [2026_09_08_eisdir-banner-txt-debug.md](./debug/2026_09_08_eisdir-banner-txt-debug.md): the Sep 3 mapper's `<path>/skill.md` wrap left directories on the new dest paths (an `assets/banner.txt/` directory holding the banner); `--force` then wrote through the directory and hit `EISDIR`. The fix: `checkCollision` deletes a leftover dest directory when `--force` is set; flat skills move to the generic `<plugin>-<skill>/SKILL.md` layout; a shared `mistral-skill-frontmatter` module emits `name`, `description`, `user-invocable` for both the build and the install paths.
43
+
- Vibe rejected the built SKILL.md, root cause and fix in [2026_09_08_vibe-skill-description-debug.md](./debug/2026_09_08_vibe-skill-description-debug.md): `parseYamlLike` dropped every CRLF key, so the built file carried only `user-invocable` and a rewritten `name`, while Vibe requires `description`. The fix: `parseFrontmatter` splits on `/\r?\n/`, the same class of fix as the catalog summarizer's. Rebuild verified 47/47 SKILL.md carrying `description`.
44
44
- The remaining commits of this phase keep the flat-build output out of the way: lefthook's markdown-links checker skips `.vibe/` and `.gitignore` ignores it; that is the flat-build output directory.
0 commit comments