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

package/CHANGELOG.md

@@ -1,5 +1,30 @@
 # Changelog
 
+## v0.7.3 (2026-07-28)
+
+Patch release that fixes Linear issue-relation updates for OAuth users.
+
+### Bug Fixes
+- **Enable issue relations with default OAuth**: New OAuth grants now request Linear's `write` scope, allowing `linear_issue update` to set `blockedBy`, `blocking`, `relatedTo`, and `duplicateOf` relations.
+- **Explain legacy-token remediation**: Relation updates rejected because `write` is missing now direct OAuth users to run `pi-linear-tools auth login`; API-key users are told to use a key with write access.
+
+### Tests
+- Added regression coverage for the default OAuth authorization URL scope and issue-relation scope guidance.
+
+### Upgrade note
+Existing OAuth tokens cannot gain the newly requested scope automatically. Re-authenticate with `pi-linear-tools auth login` before creating issue relations.
+
+## 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.3",
   "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-oauth.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/auth/constants.js

@@ -5,5 +5,10 @@
 // Linear OAuth application client ID
 export const OAUTH_CLIENT_ID = 'a3e177176c6697611367f1a2405d4a34';
 
-// OAuth scopes - minimal required scopes
-export const OAUTH_SCOPES = ['read', 'issues:create', 'comments:create'];
+// OAuth scopes - minimal required scopes.
+// `write` is included so issue relations (blockedBy / blocking / relatedTo / duplicateOf) work:
+// those go through the `issueRelationCreate` mutation, which Linear gates behind the `write`
+// scope, whereas `issueUpdate` (title/description/state/assignee/parentId) is permitted under
+// `issues:create`. Without `write`, the `linear_issue` tool accepts the relation params but
+// Linear rejects them with "Invalid scope: `write` required". See upstream issue #27.
+export const OAUTH_SCOPES = ['read', 'issues:create', 'comments:create', 'write'];

package/src/error-hints.js

@@ -22,3 +22,30 @@ export function withMilestoneScopeHint(error) {
 
   return error;
 }
+
+/**
+ * Adds remediation guidance when an issue-relation mutation is rejected because
+ * an existing credential was issued before the default OAuth scopes included `write`.
+ *
+ * @param {Error} error - The error to wrap
+ * @param {object} patch - Requested issue update fields
+ * @returns {Error} The original error or a new error with additional hint
+ */
+export function withIssueRelationScopeHint(error, patch = {}) {
+  const message = String(error?.message || error || 'Unknown error');
+  const relationFields = ['blockedBy', 'blocking', 'relatedTo', 'duplicateOf'];
+  const requestsRelation = relationFields.some((field) => {
+    const value = patch[field];
+    return Array.isArray(value) ? value.length > 0 : String(value || '').trim() !== '';
+  });
+
+  if (requestsRelation && /invalid scope/i.test(message) && /write/i.test(message)) {
+    return new Error(
+      `${message}\nHint: Issue relations require Linear's write scope. ` +
+      `If you authenticated with OAuth before this scope was added, run ` +
+      '`pi-linear-tools auth login` to re-authenticate. API-key users need a key with write access.'
+    );
+  }
+
+  return error;
+}

package/src/handlers.js

@@ -46,6 +46,7 @@ import {
   withHandlerErrorHandling,
   getViewer,
 } from './linear.js';
+import { withIssueRelationScopeHint } from './error-hints.js';
 import { debug } from './logger.js';
 
 function toTextResult(text, details = {}) {
@@ -804,7 +805,12 @@ export async function executeIssueUpdate(client, params) {
     projectMilestoneId: updatePatch.projectMilestoneId,
   });
 
-  const result = await updateIssue(client, issue, updatePatch);
+  let result;
+  try {
+    result = await updateIssue(client, issue, updatePatch);
+  } catch (error) {
+    throw withIssueRelationScopeHint(error, updatePatch);
+  }
 
   const friendlyChanges = result.changed.map((field) => {
     if (field === 'stateId') return 'state';

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;