@fink-andreas/pi-linear-tools 0.7.1 → 0.7.2

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## v0.7.2 (2026-07-21)
+
+Patch release for Pi package rename compatibility and terminal-safe fallback rendering.
+
+### Bug Fixes
+- **Support renamed Pi packages**: The extension now supports both legacy `@mariozechner/*` imports and the current `@earendil-works/*` package scope, including globally installed Pi paths.
+- **Prevent CJK fallback-rendering crashes**: Plain-text fallback output now truncates by terminal columns, preventing over-width lines from CJK/non-ASCII text and tabs from triggering Pi TUI's width guard.
+
+### Tests
+- Added fallback-renderer regression coverage for CJK text, tabs, unavailable Markdown dependencies, and Markdown rendering failures.
+
 ## v0.7.1 (2026-06-09)
 
 Patch release for issue creation milestone assignment and configuration text cleanup.

package/extensions/pi-linear-tools.js

@@ -12,7 +12,11 @@ async function importPiCodingAgent() {
   try {
     return await import('@mariozechner/pi-coding-agent');
   } catch {
-    return importFromPiRoot('dist/index.js');
+    try {
+      return await import('@earendil-works/pi-coding-agent');
+    } catch {
+      return importFromPiRoot('dist/index.js');
+    }
   }
 }
 
@@ -20,8 +24,16 @@ async function importPiTui() {
   try {
     return await import('@mariozechner/pi-tui');
   } catch {
-    // pi-tui is a dependency of pi-coding-agent and may be nested under it
-    return importFromPiRoot('node_modules/@mariozechner/pi-tui/dist/index.js');
+    try {
+      return await import('@earendil-works/pi-tui');
+    } catch {
+      // pi-tui is a dependency of pi-coding-agent and may be nested under it
+      try {
+        return await importFromPiRoot('node_modules/@mariozechner/pi-tui/dist/index.js');
+      } catch {
+        return importFromPiRoot('node_modules/@earendil-works/pi-tui/dist/index.js');
+      }
+    }
   }
 }
 
@@ -602,33 +614,56 @@ async function executeToolSafely(operationLabel, operation, options = {}) {
   }
 }
 
-function renderMarkdownResult(result, _options, _theme) {
+// Conservative column width used only by the plain-text fallback renderer:
+// printable ASCII counts as 1 column, tabs as 3 (matching pi-tui), and every
+// other code point as 2. This overestimates narrow non-ASCII characters but
+// never underestimates terminal display width, so rendered lines stay within
+// the terminal width and pi-tui's over-width guard never fires.
+function fallbackColumnWidth(codePoint) {
+  if (codePoint === 0x09) return 3;
+  return codePoint >= 0x20 && codePoint <= 0x7e ? 1 : 2;
+}
+
+export function truncateLineToColumns(line, maxColumns) {
+  let columns = 0;
+  let end = 0;
+  for (const char of line) {
+    const width = fallbackColumnWidth(char.codePointAt(0));
+    if (columns + width > maxColumns) break;
+    columns += width;
+    end += char.length;
+  }
+  return line.slice(0, end);
+}
+
+export function createPlainTextFallbackRenderer(text) {
+  const lines = text.split('\n');
+  return {
+    render: (width) => lines.map((line) => (width ? truncateLineToColumns(line, width) : line)),
+    invalidate: () => {},
+  };
+}
+
+export function renderMarkdownResult(result, _options, _theme, renderer = { Markdown, Text, getMarkdownTheme }) {
   const text = result.content?.[0]?.text || '';
+  const { Markdown: markdown, Text: textComponent, getMarkdownTheme: getTheme } = renderer;
 
   // Fall back to plain text if markdown packages not available
-  if (!Markdown || !getMarkdownTheme) {
-    const lines = text.split('\n');
-    return {
-      render: (width) => lines.map((line) => (width && line.length > width ? line.slice(0, width) : line)),
-      invalidate: () => {},
-    };
+  if (!markdown || !getTheme) {
+    return createPlainTextFallbackRenderer(text);
   }
 
   // Return Markdown component directly - the TUI will call its render() method
   try {
-    const mdTheme = getMarkdownTheme();
-    return new Markdown(text, 0, 0, mdTheme, _theme ? { color: (t) => _theme.fg('toolOutput', t) } : undefined);
+    const mdTheme = getTheme();
+    return new markdown(text, 0, 0, mdTheme, _theme ? { color: (t) => _theme.fg('toolOutput', t) } : undefined);
   } catch (error) {
     // If markdown rendering fails for any reason, show a visible error so we can diagnose.
     const msg = `[pi-linear-tools] Markdown render failed: ${String(error?.message || error)}`;
-    if (Text) {
-      return new Text((_theme ? _theme.fg('error', msg) : msg) + `\n\n` + text, 0, 0);
+    if (textComponent) {
+      return new textComponent((_theme ? _theme.fg('error', msg) : msg) + `\n\n` + text, 0, 0);
     }
-    const lines = (msg + '\n\n' + text).split('\n');
-    return {
-      render: (width) => lines.map((line) => (width && line.length > width ? line.slice(0, width) : line)),
-      invalidate: () => {},
-    };
+    return createPlainTextFallbackRenderer(msg + '\n\n' + text);
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fink-andreas/pi-linear-tools",
-  "version": "0.7.1",
+  "version": "0.7.2",
   "description": "Pi extension with Linear SDK tools and configuration commands",
   "type": "module",
   "engines": {
@@ -27,7 +27,7 @@
   },
   "scripts": {
     "start": "node index.js",
-    "test": "node tests/test-package-manifest.js && node tests/test-extension-registration.js && node tests/test-issue-comment-result.js && node tests/test-issue-priority.js && node tests/test-settings.js && node tests/test-issue-download.js && node tests/test-api-usage-caching.js && node tests/test-issue-create-milestone.js && node tests/test-assignee-update.js && node tests/test-rate-limit-fallback-update.js && node tests/test-full-assignee-flow.js && node tests/test-branch-param.js && node tests/test-team-filter.js && node tests/test-project-crud.js && node tests/test-project-lifecycle.js && node tests/test-sync-doc.js && node tests/test-issue-activity.js",
+    "test": "node tests/test-package-manifest.js && node tests/test-extension-registration.js && node tests/test-render-fallback.js && node tests/test-issue-comment-result.js && node tests/test-issue-priority.js && node tests/test-settings.js && node tests/test-issue-download.js && node tests/test-api-usage-caching.js && node tests/test-issue-create-milestone.js && node tests/test-assignee-update.js && node tests/test-rate-limit-fallback-update.js && node tests/test-full-assignee-flow.js && node tests/test-branch-param.js && node tests/test-team-filter.js && node tests/test-project-crud.js && node tests/test-project-lifecycle.js && node tests/test-sync-doc.js && node tests/test-issue-activity.js",
     "dev:sync-local-extension": "node scripts/dev-sync-local-extension.mjs",
     "release:check": "npm test && npm pack --dry-run"
   },

package/src/shared.js

@@ -18,7 +18,7 @@ export function isPiCodingAgentRoot(dir) {
   if (!fs.existsSync(pkgPath)) return false;
   try {
     const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf8'));
-    return pkg?.name === '@mariozechner/pi-coding-agent';
+    return pkg?.name === '@mariozechner/pi-coding-agent' || pkg?.name === '@earendil-works/pi-coding-agent';
   } catch {
     return false;
   }
@@ -50,9 +50,11 @@ export function findPiCodingAgentRoot() {
   {
     const binDir = path.dirname(entry);
     const prefix = path.resolve(binDir, '..');
-    const candidate = path.join(prefix, 'lib', 'node_modules', '@mariozechner', 'pi-coding-agent');
-    if (isPiCodingAgentRoot(candidate)) {
-      return candidate;
+    for (const scope of ['@mariozechner', '@earendil-works']) {
+      const candidate = path.join(prefix, 'lib', 'node_modules', scope, 'pi-coding-agent');
+      if (isPiCodingAgentRoot(candidate)) {
+        return candidate;
+      }
     }
   }
 
@@ -60,6 +62,9 @@ export function findPiCodingAgentRoot() {
   for (const candidate of [
     '/usr/local/lib/node_modules/@mariozechner/pi-coding-agent',
     '/usr/lib/node_modules/@mariozechner/pi-coding-agent',
+    '/usr/local/lib/node_modules/@earendil-works/pi-coding-agent',
+    '/usr/lib/node_modules/@earendil-works/pi-coding-agent',
+    '/opt/homebrew/lib/node_modules/@earendil-works/pi-coding-agent',
   ]) {
     if (isPiCodingAgentRoot(candidate)) {
       return candidate;