@brainervirus/workit-core 2.0.2 → 2.1.1

package/src/core/init.ts

@@ -2,12 +2,9 @@ import fs from "node:fs";
 import path from "node:path";
 import { spawnSync } from "node:child_process";
 import { configDir, isConfigObject } from "./config";
-import { PLUGIN_ROOT } from "./scripts";
 import { writeFileExclusive } from "./safe-write";
-import { resolveWorkspace, workspacesPath } from "./workspaces";
 import { applyWorkspaceBranchPolicy } from "./setup";
-import { vcsConfig, vcsVerifyToken } from "./vcs-config";
-import { youTrackTokenCreateUrl, youTrackVerifyToken } from "./youtrack";
+import { youTrackTokenCreateUrl } from "./youtrack";
 
 const TOKEN_PLACEHOLDER = "YOUR_TOKEN_HERE";
 
@@ -22,12 +19,6 @@ const readJson = (p: string): Record<string, any> | null => {
   }
 };
 
-const isPlaceholder = (text: string): boolean =>
-  text === TOKEN_PLACEHOLDER || text.startsWith(TOKEN_PLACEHOLDER);
-
-const modeOk = (p: string): boolean =>
-  process.platform === "win32" || (fs.statSync(p).mode & 0o777) === 0o600;
-
 /** Provider for a fresh vcs.json: explicit env, else origin remote, else null
  *  (no silent default — a tool for everyone must not assume a provider). */
 export const resolveInitProvider = (cwd?: string): string | null => {
@@ -50,234 +41,6 @@ export const resolveInitProvider = (cwd?: string): string | null => {
   return null;
 };
 
-const resolvePath = (p: string): string => {
-  try {
-    return fs.realpathSync(p);
-  } catch {
-    return path.resolve(p);
-  }
-};
-
-/** Port of scripts/init/status.sh — filesystem init state. */
-export function initStatusData(configDirPath = configDir()): Record<string, any> {
-  const ytJson = path.join(configDirPath, "youtrack.json");
-  const vcsJson = path.join(configDirPath, "vcs.json");
-  const items: Array<Record<string, any>> = [];
-
-  let youtrackConfig: Record<string, any> | null = null;
-  let youtrackTokenCreate: Record<string, any> | null = null;
-  const ytParsed = readJson(ytJson);
-  if (ytParsed) {
-    const base = String(ytParsed.baseUrl ?? "").replace(/\/+$/, "");
-    const meeting = String(ytParsed.meetingIssue ?? "");
-    const tokenFile = String(ytParsed.tokenFile ?? "");
-    const meetingIssues = [];
-    const rawMeetings = (ytParsed.meetingIssues ?? {}) as Record<string, any>;
-    if (rawMeetings && typeof rawMeetings === "object") {
-      for (const [key, item] of Object.entries(rawMeetings)) {
-        const iss = String(item?.issue ?? "");
-        meetingIssues.push({
-          key,
-          issue: iss,
-          label: item?.label ?? iss,
-          workItemText: item?.workItemText ?? "Reuniones",
-          url: item?.url ?? (base && iss ? `${base}/issue/${iss}` : null),
-        });
-      }
-    }
-    const expanded = tokenFile ? path.resolve(tokenFile) : null;
-    const resolvedTokenFile =
-      expanded && fs.existsSync(expanded)
-        ? resolvePath(expanded)
-        : expanded
-          ? path.isAbsolute(expanded)
-            ? expanded
-            : path.resolve(configDirPath, expanded)
-          : null;
-    youtrackConfig = {
-      config_edit_path: resolvePath(ytJson),
-      baseUrl: base,
-      meetingIssue: meeting,
-      meetingIssues,
-      meetingIssueUrl: base && meeting ? `${base}/issue/${meeting}` : null,
-      defaultMention: ytParsed.defaultMention,
-      timezone: ytParsed.timezone,
-      locale: ytParsed.locale,
-      tokenFile: resolvedTokenFile,
-      tokenDefaults: ytParsed.tokenDefaults,
-      timeLogging: {
-        meetings: {
-          options: meetingIssues,
-          skill: "/wk-meetings",
-          logsTime: true,
-          postsComment: false,
-        },
-        taskWork: {
-          issueSource: "active spec/plan **YouTrack:** field or --issue",
-          skill: "/wk-issue-update",
-          logsTime: true,
-          postsComment: true,
-        },
-      },
-    };
-    youtrackTokenCreate = youTrackTokenCreateUrl().data;
-    if (youtrackTokenCreate && !(youtrackConfig as Record<string, any>).error) {
-      youtrackConfig.tokenCreate = youtrackTokenCreate;
-    }
-  } else if (fs.existsSync(ytJson)) {
-    // Distinguish parse failure (legacy message, path in config_edit_path) from a
-    // parseable non-object, which gets the shared shape diagnostic with the path.
-    let parseFailed = false;
-    try {
-      JSON.parse(fs.readFileSync(ytJson, "utf8"));
-    } catch {
-      parseFailed = true;
-    }
-    youtrackConfig = {
-      config_edit_path: resolvePath(ytJson),
-      error: parseFailed ? "invalid youtrack.json" : `${resolvePath(ytJson)} is not a JSON object`,
-    };
-  }
-
-  items.push({
-    id: "youtrack_json",
-    label: "YouTrack config",
-    ok: fs.existsSync(ytJson) && youtrackConfig !== null && !("error" in youtrackConfig),
-    path: fs.existsSync(ytJson) ? resolvePath(ytJson) : path.resolve(ytJson),
-    config_edit_path: resolvePath(ytJson),
-    fix: "workit_init_apply action=youtrack_scaffold",
-  });
-
-  const ytTokenPath = youtrackConfig?.tokenFile ?? path.join(configDirPath, "youtrack.token");
-  const tokenText = fs.existsSync(ytTokenPath) ? fs.readFileSync(ytTokenPath, "utf8").trim() : "";
-  const placeholder = fs.existsSync(ytTokenPath) && isPlaceholder(tokenText);
-  const tokenOk =
-    fs.existsSync(ytTokenPath) &&
-    modeOk(ytTokenPath) &&
-    Boolean(tokenText) &&
-    !isPlaceholder(tokenText);
-
-  const youtrackTokenItem: Record<string, any> = {
-    id: "youtrack_token",
-    label: "YouTrack API token (mode 600, not placeholder)",
-    ok: tokenOk,
-    path: fs.existsSync(ytTokenPath) ? resolvePath(ytTokenPath) : path.resolve(ytTokenPath),
-    token_edit_path: fs.existsSync(ytTokenPath)
-      ? resolvePath(ytTokenPath)
-      : path.resolve(ytTokenPath),
-    placeholder,
-    fix: `Open ${resolvePath(ytTokenPath)} — replace ${TOKEN_PLACEHOLDER} with your permanent token, save, then /wk-status`,
-  };
-  if (youtrackTokenCreate) {
-    if (youtrackTokenCreate.createUrl)
-      youtrackTokenItem.token_create_url = youtrackTokenCreate.createUrl;
-    if (youtrackTokenCreate.docsUrl)
-      youtrackTokenItem.token_create_docs_url = youtrackTokenCreate.docsUrl;
-    if (youtrackTokenCreate.scopes) youtrackTokenItem.token_scopes = youtrackTokenCreate.scopes;
-    if (youtrackTokenCreate.tokenName) youtrackTokenItem.token_name = youtrackTokenCreate.tokenName;
-    if (youtrackTokenCreate.steps) youtrackTokenItem.token_create_steps = youtrackTokenCreate.steps;
-    youtrackTokenItem.token_prefill_supported = youtrackTokenCreate.prefillSupported ?? false;
-  }
-  items.push(youtrackTokenItem);
-
-  let vcsCfg: Record<string, any> | null = null;
-  const vcsParsed = readJson(vcsJson);
-  if (vcsParsed) {
-    const rawProvider = vcsParsed.provider;
-    const effective = vcsConfig("resolve", process.env.WORKFLOW_WORKSPACE_ROOT ?? process.cwd());
-    const provider =
-      effective?.ok && typeof effective.provider === "string"
-        ? effective.provider
-        : typeof rawProvider === "string" && rawProvider.trim()
-          ? rawProvider.toLowerCase()
-          : null;
-    vcsCfg = {
-      config_edit_path: resolvePath(vcsJson),
-      provider,
-      defaultTargetBranch: effective.ok
-        ? effective.defaultTargetBranch
-        : (vcsParsed.defaultTargetBranch ?? "develop"),
-      pr: vcsParsed.pr ?? {},
-      gitlab: vcsParsed.gitlab,
-      github: vcsParsed.github,
-      skill: "/wk-pr",
-      switchHint: 'Set "provider" to "gitlab" or "github" in vcs.json',
-    };
-  } else if (fs.existsSync(vcsJson)) {
-    vcsCfg = { config_edit_path: resolvePath(vcsJson), error: "invalid vcs.json" };
-  }
-
-  items.push({
-    id: "vcs_json",
-    label: "VCS config (GitLab / GitHub)",
-    ok: fs.existsSync(vcsJson) && vcsCfg !== null && !("error" in vcsCfg),
-    path: fs.existsSync(vcsJson) ? resolvePath(vcsJson) : path.resolve(vcsJson),
-    config_edit_path: resolvePath(vcsJson),
-    fix: "workit_init_apply action=vcs_scaffold",
-  });
-
-  return {
-    config_dir: configDirPath,
-    plugin_root: PLUGIN_ROOT,
-    token_placeholder: TOKEN_PLACEHOLDER,
-    youtrack_config: youtrackConfig,
-    vcs_config: vcsCfg,
-    items,
-    ready: false,
-  };
-}
-
-/** Port of scripts/init/toolkit-status.sh — filesystem + API health check. */
-export async function toolkitStatusData(configDirPath = configDir()): Promise<Record<string, any>> {
-  const status = initStatusData(configDirPath);
-  const tokenItem = status.items.find((i: Record<string, any>) => i.id === "youtrack_token") ?? {};
-  const placeholder = Boolean(tokenItem.placeholder);
-
-  const vcsCfg = (status.vcs_config ?? {}) as Record<string, any>;
-  const provider =
-    typeof vcsCfg.provider === "string" && vcsCfg.provider.trim()
-      ? vcsCfg.provider.toLowerCase()
-      : null;
-  const vcsJsonOk = Boolean(status.items.find((i: Record<string, any>) => i.id === "vcs_json")?.ok);
-
-  const verify = placeholder
-    ? { ok: false, error: "token still placeholder YOUR_TOKEN_HERE" }
-    : await youTrackVerifyToken();
-  const vcsVerify =
-    provider === null
-      ? {
-          ok: false,
-          error: "vcs provider unconfigured — set provider to gitlab or github in vcs.json",
-        }
-      : await vcsVerifyToken();
-  const youTrackHealth = ("data" in verify ? verify.data : verify) as Record<string, any>;
-
-  status.youtrack_verify = verify;
-  status.youtrack_ok = placeholder ? false : Boolean(youTrackHealth.ok);
-  status.vcs_verify = vcsVerify;
-  status.vcs_ok = vcsJsonOk && Boolean((vcsVerify as Record<string, any>).ok);
-
-  const fsReady = status.items.every((i: Record<string, any>) => i.required === false || i.ok);
-  status.ready = fsReady && Boolean(status.youtrack_ok) && (!vcsJsonOk || Boolean(status.vcs_ok));
-
-  if (placeholder) {
-    const tokenPath = tokenItem.token_edit_path ?? tokenItem.path ?? "";
-    const createUrl = tokenItem.token_create_url ?? status.youtrack_config?.tokenCreate?.createUrl;
-    status.token_edit_path = tokenPath;
-    if (createUrl) status.token_create_url = createUrl;
-    status.next_step = createUrl
-      ? "Open the YouTrack create-token URL, New token (name workit, scope YouTrack), paste into the token file, save, then run /wk-status"
-      : "Open the YouTrack token file, replace YOUR_TOKEN_HERE, save, then run /wk-status";
-  } else if (!status.youtrack_ok) {
-    status.next_step = "Fix YouTrack token or re-run /wk-init";
-  } else if (vcsJsonOk && !status.vcs_ok) {
-    status.next_step = `Run ${provider === "gitlab" ? "glab" : "gh"} auth login/switch, then /wk-status`;
-  } else if (status.ready) {
-    status.next_step = "All checks passed";
-  }
-  return status;
-}
-
 // Port of scripts/init/apply.sh write_youtrack_json — env overrides honored.
 const youtrackJsonContent = (dir: string): Record<string, any> => ({
   baseUrl: process.env.WORKFLOW_YT_BASE_URL ?? "https://enghouseamg.youtrack.cloud",
@@ -467,24 +230,6 @@ export function initApplyData(
   }
 }
 
-export function initStatus(): Record<string, any> {
-  const data = initStatusData();
-  const workspaces: Record<string, any> = { path: workspacesPath(), resolved: null };
-  try {
-    workspaces.resolved = resolveWorkspace(process.cwd());
-  } catch (error) {
-    workspaces.error = error instanceof Error ? error.message : String(error);
-  }
-  return {
-    ...data,
-    workspaces,
-  };
-}
-
-export async function toolkitStatus(): Promise<Record<string, any>> {
-  return toolkitStatusData();
-}
-
 export function initApply({
   action,
   confirmed,

package/src/core/policy-resolver.ts

@@ -38,51 +38,6 @@ export type ResolverPreferences = {
   thorough?: boolean;
 };
 
-/** Size facts for deterministic spec triage (op1 with override). */
-export type TriageFacts = {
-  /** New or changed observable behavior. */
-  newOrChangedBehavior: boolean;
-  /** Open ambiguity needing user resolution. */
-  ambiguityOpen: boolean;
-  /** Cross-package/host contract, auth/data/security surface, or irreversible migration. */
-  crossContract: boolean;
-  /** Subsystems touched. */
-  subsystems: number;
-  /** Packages touched (a 2-package change is Large even in one subsystem). */
-  packagesTouched: number;
-  /** Estimated implementation steps. */
-  steps: number;
-};
-
-export type TriageTier = "spec-plan" | "plan-only" | "neither";
-
-/** Large → spec + full plan; Medium → compact plan-only; Small → neither.
- * Step count alone never escalates: a long but known, single-subsystem run
- * stays plan-only. The lead may re-tier with a reason recorded in progress
- * (override). */
-export const triageTier = (facts: TriageFacts): TriageTier => {
-  if (
-    facts.newOrChangedBehavior ||
-    facts.ambiguityOpen ||
-    facts.crossContract ||
-    facts.subsystems >= 3 ||
-    facts.packagesTouched >= 2
-  )
-    return "spec-plan";
-  if (facts.steps >= 2) return "plan-only";
-  return "neither";
-};
-
-/** Tier → assessor signal values (durableAgreementNeeded, coordinationPlanNeeded). */
-export const triageSignals = (
-  tier: TriageTier,
-): { durableAgreementNeeded: boolean; coordinationPlanNeeded: boolean } =>
-  tier === "spec-plan"
-    ? { durableAgreementNeeded: true, coordinationPlanNeeded: true }
-    : tier === "plan-only"
-      ? { durableAgreementNeeded: false, coordinationPlanNeeded: true }
-      : { durableAgreementNeeded: false, coordinationPlanNeeded: false };
-
 export type ResolverInput = {
   intent: Intent;
   assessment: Assessment;

package/src/core/registration.ts

@@ -31,10 +31,9 @@ export function isWorkitPlugin(value: unknown): boolean {
   const s = String(value).replaceAll("\\", "/");
   const url = s.startsWith("file://") || s.startsWith("git+file://");
   const pkgPath =
-    s.includes("/packages/workit-opencode/") ||
-    s.includes("/packages/workit-cursor/") ||
-    s.includes("/node_modules/@brainervirus/workit-opencode/") ||
-    s.includes("/node_modules/@brainervirus/workit-cursor/");
+    /\/(?:packages\/workit-(?:opencode|cursor)|node_modules\/@brainervirus\/workit-(?:opencode|cursor))(?:\/|$)/.test(
+      s,
+    );
   return (
     named(s, "workflow-toolkit") ||
     named(s, "workflow-toolkit-opencode") ||

package/src/core/repo-context.ts

@@ -7,7 +7,7 @@ import { gitRevisionParts, resolveGitRevision } from "../core";
 // Ports of scripts/_shared/common.sh + the four maintained context generators
 // (pr-ready-context.sh, changelog-context.sh, docs-refresh-context.sh,
 // release-notes-context.sh). Each generator returns the same stdout text shape
-// the shell produced (## sections parsed by parse-sections.ts).
+// the shell produced (## sections).
 
 export type ContextResult = { stdout: string; stderr: string; exitCode: number; cwd: string };
 

package/src/core/rules.ts

@@ -1,4 +1,4 @@
-import { existsSync, mkdirSync, readFileSync, readdirSync, writeFileSync } from "node:fs";
+import { existsSync, readFileSync, readdirSync, writeFileSync } from "node:fs";
 import path from "node:path";
 import { configDir } from "./config";
 
@@ -41,71 +41,11 @@ export const parseRule = (markdown: string): CanonicalRule | { error: string } =
   return { name, description, platforms, body: fm[2].trim() + "\n" };
 };
 
-export const listRules = (): { name: string; platforms: string[]; source: "config" | "repo" }[] => {
-  const result: { name: string; platforms: string[]; source: "config" | "repo" }[] = [];
-  const dir = rulesDir();
-  if (existsSync(dir)) {
-    for (const entry of readdirSync(dir)) {
-      const file = path.join(dir, entry, "rule.md");
-      if (!existsSync(file)) continue;
-      const parsed = parseRule(readFileSync(file, "utf8"));
-      if ("error" in parsed) continue;
-      result.push({ name: parsed.name, platforms: parsed.platforms, source: "config" });
-    }
-  }
-  return result;
-};
-
-export const readRule = (
-  name: string,
-): { source: "config" | "repo" | "missing"; rule: CanonicalRule } | { error: string } => {
-  const file = path.join(rulesDir(), name, "rule.md");
-  if (existsSync(file)) {
-    const parsed = parseRule(readFileSync(file, "utf8"));
-    if ("error" in parsed) return { error: parsed.error };
-    return { source: "config", rule: parsed };
-  }
-  return { source: "missing", rule: { name, description: "", platforms: [], body: "" } };
-};
-
 const RULE_NAME_RE = /^[a-z0-9][a-z0-9-]*$/;
 
-export const writeRule = (
-  rule: CanonicalRule,
-  confirmed: boolean,
-): { ok: true; path: string } | { ok: false; error: string } => {
-  if (!confirmed) return { ok: false, error: "confirmed: true required" };
-  if (!RULE_NAME_RE.test(rule.name))
-    return { ok: false, error: `invalid rule name: ${JSON.stringify(rule.name)}` };
-  const dir = path.join(rulesDir(), rule.name);
-  mkdirSync(dir, { recursive: true });
-  const file = path.join(dir, "rule.md");
-  const md = `---\nname: ${rule.name}\ndescription: ${rule.description}\nplatforms: [${rule.platforms.join(", ")}]\n---\n${rule.body}`;
-  writeFileSync(file, md, "utf8");
-  return { ok: true, path: file };
-};
-
 export const compileRuleCursor = (rule: CanonicalRule): string =>
   `---\ndescription: ${rule.description}\nalwaysApply: true\n---\n\n${rule.body}`;
 
-export const compileRuleOpenCode = (rule: CanonicalRule): string =>
-  `## ${rule.name}\n\n${rule.body}`;
-
-export const compiledOpenCodeSections = (): string => {
-  const sections: string[] = [];
-  const dir = rulesDir();
-  if (existsSync(dir)) {
-    for (const entry of readdirSync(dir)) {
-      const file = path.join(dir, entry, "rule.md");
-      if (!existsSync(file)) continue;
-      const parsed = parseRule(readFileSync(file, "utf8"));
-      if ("error" in parsed || !parsed.platforms.includes("opencode")) continue;
-      sections.push(compileRuleOpenCode(parsed));
-    }
-  }
-  return sections.join("\n\n");
-};
-
 export const writeCompiledCursorRules = (targetDir: string): string[] => {
   const written: string[] = [];
   const dir = rulesDir();