@xzzpig/pi-starline 0.1.0 → 0.2.0

package/extensions/starline/mouse/selection-state.ts

@@ -1,50 +1,28 @@
 import { basename } from "node:path";
 
 /**
- * The one piece of selection state Starline owns.
+ * The selection hint is derived state, not owned state.
  *
- * Pi holds the anchor, the focus and the granularity; this holds only whether a
- * released selection is still waiting to be copied. Keeping it this small is
- * what the 0.2.0 controller could not do, because back then Pi had no selection
- * at all.
+ * Pi 0.84.4 owns "select without copying" (`fullscreenCopyOnSelect`), so
+ * Starline no longer intercepts the release to arm a pending copy — the hint
+ * is computed on demand from the live renderer instead: an active selection
+ * that the renderer is not auto-copying. `selectionHintText` is the pure
+ * rendering of that state; `index.ts` gathers the pieces.
  */
-export class SelectionPendingState {
-	private characters = 0;
-	/**
-	 * `keyText("app.editor.external")`, resolved at arm time, when the pending
-	 * selection is the editor's. Editor selections cannot grow past the visible
-	 * window (there is no drag-scroll), so the hint points at the external
-	 * editor as the way to act on the whole draft.
-	 */
-	private externalEditorKey: string | undefined;
-
-	arm(characters: number, externalEditorKey?: string): void {
-		this.characters = characters > 0 ? characters : 0;
-		this.externalEditorKey = this.characters > 0 ? externalEditorKey : undefined;
-	}
-
-	clear(): void {
-		this.characters = 0;
-		this.externalEditorKey = undefined;
-	}
-
-	get pending(): { characters: number; externalEditorKey?: string } | undefined {
-		return this.characters > 0
-			? { characters: this.characters, externalEditorKey: this.externalEditorKey }
-			: undefined;
-	}
-}
-
 export function selectionHintText(
-	state: SelectionPendingState,
+	characters: number,
+	copyKey: string,
+	externalEditorKey: string | undefined,
 	editorName: string | null = null,
 ): string | null {
-	const pending = state.pending;
-	if (!pending) return null;
-	const noun = pending.characters === 1 ? "character" : "characters";
-	const base = `${pending.characters} ${noun} selected, ctrl+c to copy`;
-	return pending.externalEditorKey
-		? `${base} ⋅ ${pending.externalEditorKey} to edit in ${editorName ?? "$EDITOR"}`
+	if (characters <= 0) return null;
+	const noun = characters === 1 ? "character" : "characters";
+	const base =
+		copyKey.length > 0
+			? `${characters} ${noun} selected, ${copyKey} to copy`
+			: `${characters} ${noun} selected`;
+	return externalEditorKey
+		? `${base} ⋅ ${externalEditorKey} to edit in ${editorName ?? "$EDITOR"}`
 		: base;
 }
 

package/extensions/starline/mouse/transcript-copy.ts

@@ -2,7 +2,7 @@
  * Strips decoration out of a transcript selection before it reaches the
  * clipboard.
  *
- * Pi's `copySelectionToClipboard` copies *rendered* rows. In the transcript
+ * Pi's `copyActiveSelectionToClipboard` copies *rendered* rows. In the transcript
  * those rows carry chrome that two components paint around their content:
  *
  * - Starline's own user messages (`user-message.ts`): a full-width `─` rule

package/extensions/starline/prototype-patch-registry.ts

@@ -18,8 +18,7 @@ type PrototypePatchAdapter =
 	| "mouse-viewport-input"
 	| "mouse-wheel"
 	| "mouse-selection-event"
-	| "mouse-copy"
-	| "mouse-word-selection";
+	| "mouse-copy";
 
 type PrototypeMethodName = string;
 

package/extensions/starline/ui.ts

@@ -143,7 +143,7 @@ function getEditorChromeWidths(config: PolishedTuiConfig, uiTheme: Theme, reset:
  */
 function composeRightStatus(vimStatus: string | undefined, uiTheme: Theme): string | undefined {
 	// A live selection hint already spells the external editor out ("N
-	// characters selected, ctrl+c to copy ⋅ ctrl+g to edit in $EDITOR"), so it
+	// characters selected, ctrl+x to copy ⋅ ctrl+g to edit in $EDITOR"), so it
 	// takes precedence; the always-on hint only speaks when nothing else does.
 	const hint = composeHints(
 		pasteExpandHintText(),

package/package.json

@@ -1,60 +1,59 @@
 {
-	"name": "@xzzpig/pi-starline",
-	"version": "0.1.0",
-	"description": "A Starship-inspired statusline and Opencode-style TUI for Pi.",
-	"type": "module",
-	"license": "MIT",
-	"repository": {
-		"type": "git",
-		"url": "git+https://github.com/Andy8647/pi-starline.git"
-	},
-	"homepage": "https://github.com/Andy8647/pi-starline#readme",
-	"bugs": {
-		"url": "https://github.com/Andy8647/pi-starline/issues"
-	},
-	"keywords": [
-		"pi-package",
-		"pi-extension",
-		"pi-theme",
-		"starship",
-		"tui"
-	],
-	"files": [
-		"extensions"
-	],
-	"scripts": {
-		"lint": "biome check .",
-		"typecheck": "tsc --noEmit",
-		"test": "vitest run",
-		"pi:dev": "${PI_BIN:-/opt/homebrew/bin/pi} --no-extensions -e ./extensions/starline/index.ts",
-		"pi:install-local": "${PI_BIN:-/opt/homebrew/bin/pi} install ./ -l",
-		"fmt": "biome format --write .",
-		"fmt:check": "biome format --check .",
-		"verify": "npm run lint && npm run typecheck && npm run test",
-		"pack:check": "npm pack --dry-run",
-		"prepublishOnly": "npm run verify && npm run pack:check"
-	},
-	"peerDependencies": {
-		"@earendil-works/pi-ai": ">=0.80.3",
-		"@earendil-works/pi-coding-agent": ">=0.80.3",
-		"@earendil-works/pi-tui": ">=0.80.3"
-	},
-	"pi": {
-		"extensions": [
-			"./extensions"
-		],
-		"image": "https://raw.githubusercontent.com/Andy8647/pi-starline/main/assets/starline.png"
-	},
-	"publishConfig": {
-		"access": "public"
-	},
-	"devDependencies": {
-		"@biomejs/biome": "^2.5.0",
-		"@earendil-works/pi-ai": "^0.82.1",
-		"@earendil-works/pi-coding-agent": "^0.84.1",
-		"@earendil-works/pi-tui": "^0.84.1",
-		"@types/node": "^26.0.0",
-		"typescript": "^7.0.2",
-		"vitest": "^4.1.8"
-	}
-}
+  "name": "@xzzpig/pi-starline",
+  "version": "0.2.0",
+  "description": "A Starship-inspired statusline and Opencode-style TUI for Pi.",
+  "type": "module",
+  "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/Andy8647/pi-starline.git"
+  },
+  "homepage": "https://github.com/Andy8647/pi-starline#readme",
+  "bugs": {
+    "url": "https://github.com/Andy8647/pi-starline/issues"
+  },
+  "keywords": [
+    "pi-package",
+    "pi-extension",
+    "pi-theme",
+    "starship",
+    "tui"
+  ],
+  "files": [
+    "extensions"
+  ],
+  "peerDependencies": {
+    "@earendil-works/pi-ai": ">=0.84.4",
+    "@earendil-works/pi-coding-agent": ">=0.84.4",
+    "@earendil-works/pi-tui": ">=0.84.4"
+  },
+  "pi": {
+    "extensions": [
+      "./extensions"
+    ],
+    "image": "https://raw.githubusercontent.com/Andy8647/pi-starline/main/assets/starline.png"
+  },
+  "publishConfig": {
+    "access": "public"
+  },
+  "devDependencies": {
+    "@biomejs/biome": "^2.5.0",
+    "@earendil-works/pi-ai": "^0.84.4",
+    "@earendil-works/pi-coding-agent": "^0.84.4",
+    "@earendil-works/pi-tui": "^0.84.4",
+    "@types/node": "^26.0.0",
+    "typescript": "^7.0.2",
+    "vitest": "^4.1.8"
+  },
+  "scripts": {
+    "lint": "biome check .",
+    "typecheck": "tsc --noEmit",
+    "test": "vitest run",
+    "pi:dev": "${PI_BIN:-/opt/homebrew/bin/pi} --no-extensions -e ./extensions/starline/index.ts",
+    "pi:install-local": "${PI_BIN:-/opt/homebrew/bin/pi} install ./ -l",
+    "fmt": "biome format --write .",
+    "fmt:check": "biome format --check .",
+    "verify": "npm run lint && npm run typecheck && npm run test",
+    "pack:check": "npm pack --dry-run"
+  }
+}

package/extensions/starline/mouse/word-select.ts

@@ -1,145 +0,0 @@
-/**
- * Which columns a double click on one line should select.
- *
- * Ported from `getWordSelection` in pi commit `56419be51`
- * (`feat/word-selection-path-chars`, `packages/tui/src/tui-alt-screen.ts`),
- * filed upstream as issue #7746. The installed Pi build's own
- * `getWordSelection` predates that patch, so this substitutes Pi's whole
- * word-selection rule rather than layering path handling on top of it — see
- * `index.ts` for where that substitution happens and why it's the intended
- * scope, not creep.
- *
- * This is a pure function of one already-plain (no ANSI) line and a terminal
- * column; the caller in `index.ts` is what reads Pi's live source line and
- * strips escape sequences before calling in, matching the split
- * `getWordSelection`/`getSelectionSourceLine` already have upstream.
- */
-import { visibleWidth } from "@earendil-works/pi-tui";
-
-const wordSegmenter = new Intl.Segmenter(undefined, { granularity: "word" });
-
-/**
- * Characters that join two words into one for double-click selection.
- *
- * `Intl.Segmenter` already keeps `_` and `.` inside a word, so `foo_bar` and
- * `file.ts` survive on their own. It breaks on `/` and `-`, which is what
- * takes a path or a kebab-case identifier apart. Terminals keep these inside
- * a word so paths and URLs select whole; this set is what makes fullscreen's
- * own double-click agree with them.
- */
-const WORD_JOINERS = new Set(["/", "-"]);
-
-type WordSegment = {
-	start: number;
-	end: number;
-	isWordLike: boolean;
-	isJoiner: boolean;
-	isSpace: boolean;
-};
-
-/** `segments[index]`, guaranteed — every call site here only ever passes an index it just found in range. */
-function segmentAt(segments: readonly WordSegment[], index: number): WordSegment {
-	const segment = segments[index];
-	if (segment === undefined) {
-		throw new RangeError(`word-select: no segment at index ${index}`);
-	}
-	return segment;
-}
-
-function collectWordSegments(line: string): WordSegment[] {
-	const segments: WordSegment[] = [];
-	let start = 0;
-	for (const segment of wordSegmenter.segment(line)) {
-		const text = segment.segment;
-		const end = start + visibleWidth(text);
-		segments.push({
-			start,
-			end,
-			isWordLike: segment.isWordLike === true,
-			// Joiners arrive one per segment, so `//` in a URL is two of these.
-			isJoiner: text.length > 0 && [...text].every((char) => WORD_JOINERS.has(char)),
-			isSpace: text.length > 0 && text.trim().length === 0,
-		});
-		start = end;
-	}
-	return segments;
-}
-
-/**
- * Walk back over a run of joiners that opens a word, as in `/usr` or `--flag`.
- *
- * The run only counts as part of the word when nothing but whitespace
- * precedes it, which is what separates a leading separator from an infix one.
- * In `http://a.com` the `//` is preceded by `:`, so the scheme stays out.
- */
-function openingJoinerRun(segments: WordSegment[], first: number): number {
-	let index = first;
-	while (segments[index - 1]?.isJoiner) index--;
-	if (index === first) return first;
-	const before = segments[index - 1];
-	return before === undefined || before.isSpace ? index : first;
-}
-
-/**
- * Grow a word segment across joiners that have a word on both sides, plus any
- * separator run that opens it.
- *
- * Anchoring on word-like segments is what keeps this conservative: in
- * `http://a.com/b` the `:` is not a joiner, so the run stops there instead of
- * swallowing the scheme, and a dash used as prose punctuation in `a - b` has
- * whitespace around it rather than a word.
- */
-function expandOverWordJoiners(
-	segments: WordSegment[],
-	index: number,
-): { start: number; end: number } {
-	const anchor = segmentAt(segments, index);
-	if (anchor.isJoiner) {
-		// Clicking a separator should take the word it belongs to. Prefer the
-		// word on the left so an infix `/` grows the whole path; failing that,
-		// step over the run to the word it opens, so clicking the `/` of
-		// `/usr/bin` works.
-		if (segments[index - 1]?.isWordLike && segments[index + 1]?.isWordLike) {
-			return expandOverWordJoiners(segments, index - 1);
-		}
-		if (!segments[index - 1]?.isWordLike) {
-			let next = index;
-			while (segments[next]?.isJoiner) next++;
-			if (segments[next]?.isWordLike) return expandOverWordJoiners(segments, next);
-		}
-	}
-	if (!anchor.isWordLike) return { start: anchor.start, end: anchor.end };
-
-	let first = index;
-	while (first >= 2 && segments[first - 1]?.isJoiner && segments[first - 2]?.isWordLike) {
-		first -= 2;
-	}
-	first = openingJoinerRun(segments, first);
-	// A trailing run gets no such treatment: nothing follows it to make it
-	// part of the word, and `trailing- x` reading as `trailing-` would be
-	// wrong.
-	let last = index;
-	while (segments[last + 1]?.isJoiner && segments[last + 2]?.isWordLike) {
-		last += 2;
-	}
-	return { start: segmentAt(segments, first).start, end: segmentAt(segments, last).end };
-}
-
-/**
- * The column range a double click at `column` on `line` should select, in
- * the same column units as Pi's own `SelectionRange`. `undefined` when
- * `column` lands past the end of the line — the caller calls through to Pi's
- * own selection in that case rather than inventing a range.
- */
-export function wordRangeAt(
-	line: string,
-	column: number,
-): { start: number; end: number } | undefined {
-	const segments = collectWordSegments(line);
-	for (let index = 0; index < segments.length; index++) {
-		const segment = segmentAt(segments, index);
-		if (column < segment.start || column >= segment.end) continue;
-		return expandOverWordJoiners(segments, index);
-	}
-	return undefined;
-}