From 1e996f70b8c24210d7bfb03a93eed7a8cde002a7 Mon Sep 17 00:00:00 2001 From: minhn4 Date: Thu, 6 Aug 2026 08:04:09 +0700 Subject: [PATCH] Point the office skills at ~/.claude/skills via the setup-script default The bundled setup scripts now default their skills root to ~/.claude/skills so Claude Code (not just CoDev Code) reads the office skills. codevhub still passes no -SkillsRoot, so it inherits that default and preserves the script's console-user detection under a different-account Windows elevation. Documents the destination in the office.ts header and updates the manual-command example path. Co-Authored-By: Claude Fable 5 --- src/lib/office.ts | 6 +++++- tests/lib/download.test.ts | 4 ++-- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/src/lib/office.ts b/src/lib/office.ts index 45a34b6..e6bce7d 100644 --- a/src/lib/office.ts +++ b/src/lib/office.ts @@ -15,7 +15,11 @@ import { legacyOfficeDownloadsDir, officeDownloadsDir } from "@/lib/paths.js"; // `codevhub skill office`: fetch the CoDev Office offline bundle (published by // the codev-storage MinIO backend) for this OS and run the bundled setup -// script, which installs the Office skills (DOCX, XLSX, PPTX and PDF). +// script, which installs the Office skills (DOCX, XLSX, PPTX and PDF) into the +// global ~/.claude/skills dir (read by Claude Code and CoDev Code). codevhub +// passes no skills-root — the destination is the setup script's own default, +// which also detects the signed-in user under a different-account Windows +// elevation, so codevhub must not pin a path that would defeat that. // File names are deterministic per platform — no manifest fetch — and each // bundle carries its own SHA256SUMS.txt that the setup flow can verify. // Non-interactive on purpose — the second half hands the terminal to an diff --git a/tests/lib/download.test.ts b/tests/lib/download.test.ts index 5119f84..11c7da5 100644 --- a/tests/lib/download.test.ts +++ b/tests/lib/download.test.ts @@ -429,14 +429,14 @@ describe("runSkillOffice", () => { const line = officeManualWindowsCommand("codev-office-windows-setup.ps1", [ "-SkipVerify", "-SkillsRoot", - "C:\\Users\\Van Phong\\.config\\codev\\skills", + "C:\\Users\\Van Phong\\.claude\\skills", ]); expect(line).toContain( "powershell -ExecutionPolicy Bypass -File .\\codev-office-windows-setup.ps1 -SkipVerify", ); // Space-containing paths are quoted so copy-paste survives them. expect(line).toContain( - '-SkillsRoot "C:\\Users\\Van Phong\\.config\\codev\\skills"', + '-SkillsRoot "C:\\Users\\Van Phong\\.claude\\skills"', ); });