mosaicstack/stack
1
0
Fork 1
Code Issues 212 Pull Requests 2 Actions Packages Projects Releases 15 Wiki Activity

test(skill): install linker asserts harness-home link topology

Browse Source 
The two install-linker-compatibility tests still asserted the pre-isolation
behavior (mosaic skill links planted in $HOME/.claude/skills). This branch
deliberately moved the link farm into the mosaic-owned harness homes
($MOSAIC_HOME/.claude/skills) and demoted the base-install dirs to
cleanup-only legacy targets, so the tests now assert the new topology:
the skill links appear under the harness home, foreign links in the legacy
dir are preserved, and no new mosaic link is planted in the base install.

Co-Authored-By: Claude Fable 5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01Dtdjx4Gxude9fwyLezCrhh
This commit is contained in:
mos-dt-0
2026-08-11 20:51:03 -05:00
committed by Jason Woltje
co-authored by Claude Fable 5
parent 709a23d08c
commit dd6357e670
1 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/mosaic/src/commands/skill.spec.ts
+8 -3
View File
@@ -332,9 +332,11 @@ describe('Claude skill bridge', () => {
expect(result.status, result.stderr).toBe(0); expect(result.status, result.stderr).toBe(0);
expect(readlinkSync(liveForeignLink)).toBe(liveForeignTarget); expect(readlinkSync(liveForeignLink)).toBe(liveForeignTarget);
expect(readlinkSync(danglingForeignLink)).toBe(join(mosaicHome, 'foreign-missing')); expect(readlinkSync(danglingForeignLink)).toBe(join(mosaicHome, 'foreign-missing'));
expect(readlinkSync(join(paths.claudeSkillsDir, 'missing'))).toBe( // Skills now link into the mosaic-owned harness home, never the base install.
expect(readlinkSync(join(mosaicHome, '.claude', 'skills', 'missing'))).toBe(
join(paths.mosaicSkillsDir, 'missing'), join(paths.mosaicSkillsDir, 'missing'),
); );
expect(existsSync(join(paths.claudeSkillsDir, 'missing'))).toBe(false);
}); });
it('preserves live and dangling foreign Claude symlinks while linking missing skills', () => { it('preserves live and dangling foreign Claude symlinks while linking missing skills', () => {
@@ -349,17 +351,20 @@ describe('Claude skill bridge', () => {
symlinkSync(external, liveLink); symlinkSync(external, liveLink);
symlinkSync(join(root, 'external-missing'), danglingLink); symlinkSync(join(root, 'external-missing'), danglingLink);
const mosaicHome = join(root, '.config', 'mosaic');
const result = spawnSync('bash', [LEGACY_SYNC_SCRIPT, '--link-only'], { const result = spawnSync('bash', [LEGACY_SYNC_SCRIPT, '--link-only'], {
encoding: 'utf8', encoding: 'utf8',
env: { ...process.env, HOME: root, MOSAIC_HOME: join(root, '.config', 'mosaic') }, env: { ...process.env, HOME: root, MOSAIC_HOME: mosaicHome },
}); });
expect(result.status, result.stderr).toBe(0); expect(result.status, result.stderr).toBe(0);
expect(readlinkSync(liveLink)).toBe(external); expect(readlinkSync(liveLink)).toBe(external);
expect(readlinkSync(danglingLink)).toBe(join(root, 'external-missing')); expect(readlinkSync(danglingLink)).toBe(join(root, 'external-missing'));
expect(readlinkSync(join(paths.claudeSkillsDir, 'missing'))).toBe( // Skills now link into the mosaic-owned harness home, never the base install.
expect(readlinkSync(join(mosaicHome, '.claude', 'skills', 'missing'))).toBe(
join(paths.mosaicSkillsDir, 'missing'), join(paths.mosaicSkillsDir, 'missing'),
); );
expect(existsSync(join(paths.claudeSkillsDir, 'missing'))).toBe(false);
}); });
}); });