@staff0rd/assist 0.147.0 → 0.147.2

package/claude/commands/test-cover.md

@@ -51,7 +51,7 @@ Use a behavioural, BDD-style structure:
 
 - **Outer `describe`** — the function or module under test
 - **Inner `describe("when ...")`** — groups tests that share the same setup/scenario
-- **`it("should ...")`** — asserts a single rule or behaviour
+- **`it("should ...")`** — asserts a single rule or behaviour. State specifiers (flags, syntax variants, conditions) belong in `when` blocks, never in `it` descriptions — keep assertions as bare outcomes (e.g. "should allow", "should reject")
 
 Each test follows an **Arrange, Act, Assert** pattern. Do NOT add `// arrange`, `// act`, `// assert` comments — just structure the code in that order with whitespace separating the three sections.
 

package/claude/commands/test-review.md

@@ -35,7 +35,7 @@ Assess each test file against these criteria:
 ### BDD structure and Arrange-Act-Assert
 - Does the outer `describe` name the function or module under test?
 - Do inner `describe("when ...")` blocks group tests by shared setup/scenario?
-- Do `it` blocks use `should` phrasing (e.g., `it("should return empty array when no matches")`)?
+- Do `it` blocks use `should` phrasing with bare outcomes (e.g., `it("should allow")`, `it("should reject")`)? State specifiers (flags, syntax variants, conditions) belong in `when` blocks, never in `it` descriptions.
 - Does each test follow Arrange, Act, Assert ordering (without comments labelling the sections)?
 - Are assertions minimal per test — ideally one, at most two closely related? If multiple things are asserted about the same action, are they split into separate `it` blocks under the same `describe("when ...")`?
 

package/dist/index.js

@@ -6,7 +6,7 @@ import { Command } from "commander";
 // package.json
 var package_default = {
   name: "@staff0rd/assist",
-  version: "0.147.0",
+  version: "0.147.2",
   type: "module",
   main: "dist/index.js",
   bin: {
@@ -7290,6 +7290,18 @@ async function applyExtraction(functionName, sourceFile, destPath, plan2, projec
   }
   updateImporters(functionName, sourceFile, plan2.importersToUpdate);
   if (plan2.barrel) {
+    for (const decl of plan2.barrel.getExportDeclarations()) {
+      const named = decl.getNamedExports();
+      const match = named.find((n) => n.getName() === functionName);
+      if (match) {
+        if (named.length === 1) {
+          decl.remove();
+        } else {
+          match.remove();
+        }
+        break;
+      }
+    }
     plan2.barrel.addExportDeclaration({
       moduleSpecifier: plan2.barrelRelPath,
       namedExports: [functionName]
@@ -7298,9 +7310,6 @@ async function applyExtraction(functionName, sourceFile, destPath, plan2, projec
   await project.save();
 }
 
-// src/commands/refactor/extract/buildPlan.ts
-import path29 from "path";
-
 // src/commands/refactor/extract/collectDependencies.ts
 import {
   SyntaxKind as SyntaxKind5
@@ -7628,6 +7637,26 @@ function findImporters(functionName, sourceFile, destPath, project) {
   return result;
 }
 
+// src/commands/refactor/extract/resolveBarrel.ts
+import path29 from "path";
+function resolveBarrel(functionName, sourcePath, destPath, project) {
+  const indexPath = path29.join(path29.dirname(destPath), "index.ts");
+  const barrel = project.getSourceFile(indexPath);
+  if (!barrel) return { barrel: void 0, barrelRelPath: "" };
+  const sourceRelFromBarrel = getRelativeImportPath(indexPath, sourcePath);
+  const alreadyExported = barrel.getExportDeclarations().some((decl) => {
+    const specifier = decl.getModuleSpecifierValue();
+    if (specifier !== sourceRelFromBarrel) return false;
+    const named = decl.getNamedExports();
+    return named.some((n) => n.getName() === functionName);
+  });
+  if (!alreadyExported) return { barrel: void 0, barrelRelPath: "" };
+  return {
+    barrel,
+    barrelRelPath: getRelativeImportPath(indexPath, destPath)
+  };
+}
+
 // src/commands/refactor/extract/sourceReferencesName.ts
 import { SyntaxKind as SyntaxKind10 } from "ts-morph";
 var DECLARATION_KINDS = /* @__PURE__ */ new Set([
@@ -7660,13 +7689,6 @@ function sourceReferencesName(sourceFile, functionName, extractedNames) {
 }
 
 // src/commands/refactor/extract/buildPlan.ts
-function resolveBarrel(destPath, project) {
-  const indexPath = path29.join(path29.dirname(destPath), "index.ts");
-  return {
-    barrel: project.getSourceFile(indexPath),
-    barrelRelPath: getRelativeImportPath(indexPath, destPath)
-  };
-}
 function buildPlan(functionName, sourceFile, sourcePath, destPath, project) {
   const analysis = analyseTarget(sourceFile, functionName);
   const sourceRelPath = getRelativeImportPath(destPath, sourcePath);
@@ -7687,7 +7709,7 @@ function buildPlan(functionName, sourceFile, sourcePath, destPath, project) {
       analysis.extractedNames
     ),
     importersToUpdate: analysis.target.isExported() ? findImporters(functionName, sourceFile, destPath, project) : [],
-    ...resolveBarrel(destPath, project)
+    ...resolveBarrel(functionName, sourcePath, destPath, project)
   };
 }
 
@@ -10254,7 +10276,7 @@ import { fileURLToPath as fileURLToPath7 } from "url";
 import * as fs23 from "fs";
 import * as path45 from "path";
 import chalk105 from "chalk";
-async function syncClaudeMd(claudeDir, targetBase) {
+async function syncClaudeMd(claudeDir, targetBase, options2) {
   const source = path45.join(claudeDir, "CLAUDE.md");
   const target = path45.join(targetBase, "CLAUDE.md");
   const sourceContent = fs23.readFileSync(source, "utf-8");
@@ -10266,7 +10288,7 @@ async function syncClaudeMd(claudeDir, targetBase) {
       );
       console.log();
       printDiff(targetContent, sourceContent);
-      const confirm = await promptConfirm(
+      const confirm = options2?.yes || await promptConfirm(
         chalk105.red("Overwrite existing CLAUDE.md?"),
         false
       );
@@ -10328,7 +10350,7 @@ async function sync(options2) {
   const targetBase = path47.join(os.homedir(), ".claude");
   syncCommands(claudeDir, targetBase);
   await syncSettings(claudeDir, targetBase, { yes: options2?.yes });
-  await syncClaudeMd(claudeDir, targetBase);
+  await syncClaudeMd(claudeDir, targetBase, { yes: options2?.yes });
 }
 function syncCommands(claudeDir, targetBase) {
   const sourceDir = path47.join(claudeDir, "commands");

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@staff0rd/assist",
-	"version": "0.147.0",
+	"version": "0.147.2",
 	"type": "module",
 	"main": "dist/index.js",
 	"bin": {