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

package/CHANGELOG.md

@@ -1,25 +1,46 @@
 # Changelog
 
-## v0.7.0 (2026-05-05)
+## 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.
+
+### Bug Fixes
+- **Fixed issue create milestone assignment**: `linear_issue create` now assigns project milestones when `milestone` or `projectMilestoneId` is provided.
+- **Fixed configuration text typo**: Corrected config command/help text in the pi extension.
+
+### Release Maintenance
+- Updated package metadata to `0.7.1`.
+
+## v0.7.0 (2026-06-09)
 
 Issue image fetching and file download release.
 
 ### New Features
-- **`linear_issue(action="images")`**: Fetch markdown/HTML embedded images from issue descriptions and comments. Returns images as tool `image` content (base64-encoded). Includes special auth handling for Linear upload URLs (`uploads.linear.app`).
-- **`linear_issue(action="download")`**: Download Linear issue attachments to local filesystem. Supports attachment selection by ID, title, URL, or index. Includes relative path validation, filename sanitization, and `allow_overwrite_files` config guard (default: `false`).
+- **Issue image fetching**: Added `linear_issue(action="images")` to fetch embedded markdown/HTML images from issue descriptions and comments as tool image content.
+- **Issue attachment downloads**: Added `linear_issue(action="download")` to download Linear issue attachments to validated local paths with filename sanitization and overwrite controls.
+- **Workspace-wide issue listing**: `linear_issue list` can now list across the workspace without requiring a project context.
+- **Overwrite configuration**: Added `allow_overwrite_files` config support via `/linear-tools-config` and `pi-linear-tools config`.
 
-### Performance / API Usage
-- **N+1 issue resolution optimization**: Merged N+1-per-issue lazy loads in milestone details into a single GraphQL batch query (introduced in v0.6.0).
+### Bug Fixes
+- Fixed project archive mutation so `archiveProject` archives instead of deleting projects.
+- Fixed comment rendering so submitted comment text appears in `linear_issue comment` results.
+- Fixed rate-limit display to reflect actual complexity-based limits instead of a hardcoded total.
+- Added project and project-update rate-limit pre-checks.
 
 ### Documentation
-- **Updated smoke tests**: Added coverage for `images` and `download` actions.
-
-### Bug Fixes
-- **Fixed workspace-wide issue listing (#4)**: `linear_issue list` now falls back to workspace-level listing (using `fetchIssues`) when no project is specified and cwd doesn't match a Linear project. This allows `linear_issue list assignee=me` to work without requiring a project context.
-- Fixed project archive mutation to actually archive instead of delete (INN-301).
-- Fixed `linear_issue comment` to render submitted comment text in result (INN-305).
-- Fixed rate limit display to not show hardcoded total=5000 (INN-304).
-- Fixed `linear_project` and `linear_project_update` to have rate-limit pre-check (INN-303).
+- Added smoke test coverage for image and attachment workflows.
+- Added dedicated release notes in `docs/RELEASE_NOTES_v0.7.0.md`.
 
 ## v0.6.0 (2026-04-23)
 

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');
+      }
+    }
   }
 }
 
@@ -358,7 +370,7 @@ async function runInteractiveConfigFlow(ctx, pi) {
   }
 
   if (!client) {
-    const selectedAuthMethod = await ctx.ui.select('Select authentication method', ['API Key (recommended for full functionlaity)', 'OAuth']);
+    const selectedAuthMethod = await ctx.ui.select('Select authentication method', ['API Key (recommended for full functionality)', 'OAuth']);
 
     if (!selectedAuthMethod) {
       ctx.ui.notify('Configuration cancelled', 'warning');
@@ -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);
   }
 }
 
@@ -748,11 +783,11 @@ async function registerLinearTools(pi) {
         },
         milestone: {
           type: 'string',
-          description: 'For update: milestone name/ID, or "none" to clear milestone assignment.',
+          description: 'For create/update: milestone name/ID. For create, also provide project so names can be resolved. Use "none" to clear on update.',
         },
         projectMilestoneId: {
           type: 'string',
-          description: 'Optional explicit milestone ID alias for update.',
+          description: 'Optional explicit milestone ID alias for create/update.',
         },
         subIssueOf: {
           type: 'string',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fink-andreas/pi-linear-tools",
-  "version": "0.7.0",
+  "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-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/handlers.js

@@ -25,7 +25,6 @@ import {
   fetchIssueActivity,
   formatIssueAsMarkdown,
   formatIssueActivityAsMarkdown,
-  fetchIssues,
   fetchIssuesByProject,
   fetchProjectMilestones,
   fetchMilestoneDetails,
@@ -390,24 +389,12 @@ async function startGitBranch(branchName, fromRef = 'HEAD', onBranchExists = 'sw
 export async function executeIssueList(client, params) {
   return withHandlerErrorHandling(async () => {
     let projectRef = params.project;
-    let projectMode = false;
-
-    if (projectRef) {
-      // Explicit project specified - use project-scoped listing
-      projectMode = true;
-    } else {
-      // No explicit project - try cwd fallback
-      const cwdProject = path.basename(process.cwd());
-      try {
-        const resolved = await resolveProjectRef(client, cwdProject);
-        projectRef = resolved.name; // use resolved name for display
-        projectMode = true;
-      } catch {
-        // cwd doesn't match any project - fall back to workspace-level listing
-        projectMode = false;
-      }
+    if (!projectRef) {
+      projectRef = path.basename(process.cwd());
     }
 
+    const resolved = await resolveProjectRef(client, projectRef);
+
     let assigneeId = null;
     if (params.assignee === 'me') {
       const viewer = await getViewer(client);
@@ -421,50 +408,21 @@ export async function executeIssueList(client, params) {
       teamId = team.id;
     }
 
-    const limit = params.limit || 20;
-
-    let issues;
-    let truncated;
-    let projectId = null;
-    let projectName = null;
-
-    if (projectMode) {
-      // Project-scoped listing
-      const resolved = await resolveProjectRef(client, projectRef);
-      projectId = resolved.id;
-      projectName = resolved.name;
-
-      const result = await fetchIssuesByProject(client, resolved.id, params.states || null, {
-        assigneeId,
-        teamId,
-        limit,
-      });
-      issues = result.issues;
-      truncated = result.truncated;
-    } else {
-      // Workspace-level listing (no project filter)
-      // Default to open states if none specified
-      const workspaceStates = params.states || ['Backlog', 'Triage', 'In Progress', 'In Review'];
-      const result = await fetchIssues(client, assigneeId, workspaceStates, limit);
-      issues = result.issues;
-      truncated = result.truncated;
-    }
+    const { issues, truncated } = await fetchIssuesByProject(client, resolved.id, params.states || null, {
+      assigneeId,
+      teamId,
+      limit: params.limit || 20,
+    });
 
     if (issues.length === 0) {
-      return toTextResult(
-        projectMode
-          ? `No issues found in project "${projectName}"`
-          : 'No issues found in workspace',
-        {
-          projectId,
-          projectName,
-          issueCount: 0,
-        }
-      );
+      return toTextResult(`No issues found in project "${resolved.name}"`, {
+        projectId: resolved.id,
+        projectName: resolved.name,
+        issueCount: 0,
+      });
     }
 
-    const scopeLabel = projectMode ? `project "${projectName}"` : 'workspace';
-    const lines = [`## Issues in ${scopeLabel} (${issues.length}${truncated ? '+' : ''})\n`];
+    const lines = [`## Issues in project "${resolved.name}" (${issues.length}${truncated ? '+' : ''})\n`];
 
     for (const issue of issues) {
       const stateLabel = issue.state?.name || 'Unknown';
@@ -484,8 +442,8 @@ export async function executeIssueList(client, params) {
     }
 
     return toTextResult(lines.join('\n'), {
-      projectId,
-      projectName,
+      projectId: resolved.id,
+      projectName: resolved.name,
       issueCount: issues.length,
       truncated,
     });
@@ -732,6 +690,22 @@ export async function executeIssueCreate(client, params, options = {}) {
     createInput.projectId = resolvedProject.id;
   }
 
+  if (params.projectMilestoneId !== undefined && params.projectMilestoneId !== null) {
+    createInput.projectMilestoneId = params.projectMilestoneId;
+  } else if (params.milestone !== undefined && params.milestone !== null) {
+    const milestoneRef = String(params.milestone || '').trim();
+    const clearMilestoneValues = new Set(['', 'none', 'null', 'unassigned', 'clear']);
+
+    if (!clearMilestoneValues.has(milestoneRef.toLowerCase())) {
+      if (!resolvedProject?.id) {
+        throw new Error('Missing required field: project. Provide project when assigning milestone by name during issue create, or use projectMilestoneId.');
+      }
+
+      const milestone = await resolveMilestoneRef(client, milestoneRef, resolvedProject.id);
+      createInput.projectMilestoneId = milestone.id;
+    }
+  }
+
   const issue = await createIssue(client, createInput);
 
   const identifier = issue.identifier || issue.id || 'unknown';
@@ -742,8 +716,11 @@ export async function executeIssueCreate(client, params, options = {}) {
   const stateLabel = issue.state?.name || 'Unknown';
   const assigneeLabel = issue.assignee?.displayName || 'Unassigned';
 
+  const milestoneLabel = issue.projectMilestone?.name || null;
+
   const metaParts = [`Team: ${team.name}`, `Project: ${projectLabel}`, `State: ${stateLabel}`, `Assignee: ${assigneeLabel}`];
   if (priorityLabel) metaParts.push(`Priority: ${priorityLabel}`);
+  if (milestoneLabel) metaParts.push(`Milestone: ${milestoneLabel}`);
 
   return toTextResult(
     `Created issue **${identifier}**: ${issue.title}\n${metaParts.join(' | ')}`,
@@ -755,6 +732,7 @@ export async function executeIssueCreate(client, params, options = {}) {
       project: issue.project,
       state: issue.state,
       assignee: issue.assignee,
+      projectMilestone: issue.projectMilestone,
       url: issue.url,
     }
   );

package/src/linear.js

@@ -3150,6 +3150,7 @@ export async function setIssueState(client, issueId, stateId) {
  * @param {string} [input.projectId] - Project ID
  * @param {number|string} [input.priority] - Issue priority: 0=None, 1=Urgent, 2=High, 3=Medium, 4=Low; or none/urgent/high/medium/low
  * @param {string} [input.assigneeId] - Assignee ID
+ * @param {string} [input.projectMilestoneId] - Project milestone ID
  * @param {string} [input.parentId] - Parent issue ID for sub-issues
  * @returns {Promise<Object>} Created issue
  */
@@ -3202,6 +3203,10 @@ export async function createIssue(client, input) {
       createInput.stateId = input.stateId;
     }
 
+    if (input.projectMilestoneId !== undefined) {
+      createInput.projectMilestoneId = input.projectMilestoneId;
+    }
+
     if (getRawRequest(client)) {
       const payload = await executeGraphQL(client, ISSUE_CREATE_MUTATION, { input: createInput });
       if (!payload?.issueCreate?.success) {
@@ -3245,6 +3250,9 @@ export async function createIssue(client, input) {
         state: null,
         team: null,
         project: null,
+        projectMilestone: createInput.projectMilestoneId
+          ? { id: createInput.projectMilestoneId, name: 'Unknown' }
+          : null,
         assignee: null,
       };
     }
@@ -3259,6 +3267,9 @@ export async function createIssue(client, input) {
       state: null,
       team: null,
       project: null,
+      projectMilestone: createInput.projectMilestoneId
+        ? { id: createInput.projectMilestoneId, name: 'Unknown' }
+        : null,
       assignee: null,
     };
   }, 'createIssue');

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;