@underactive/pi-topping-statusline 0.1.2 → 0.1.3

package/README.md

@@ -41,6 +41,7 @@ and apply live.
 ║    Symbols                                       ‹ nerdfont ›  ║
 ║    Border style                                   ‹ rounded ›  ║
 ║    [■] Rainbow border on max thinking                      ON  ║
+║    [■] Animate rainbow border                              ON  ║
 ║                                                                ║
 ╟─ Top Left Segment Group ───────────────────────────────────────╢
 ║    [■] Pi symbol                                           ON  ║
@@ -65,26 +66,26 @@ and apply live.
 ║    + add feed                                                  ║
 ║                                                                ║
 ║  ↑↓ move  ←→ cycle  ␣ toggle  ⏎ apply/edit  esc cancel         ║
-╚════════════════════════════════════════════════════════[ 1/28 ]╝
+╚════════════════════════════════════════════════════════[ 1/29 ]╝
 ```
 
 | Section | Settings |
 | --- | --- |
-| Global | Transparent Segments · Separator (`powerline` `powerline-thin` `slash` `pipe` `ascii`) · Symbols (`nerdfont` `unicode` `ascii` — stored in settings.json as `nerd`/`unicode`/`ascii`) · Border style (`rounded` `heavy` `double` `single`) · Rainbow border on max thinking |
+| Global | Transparent Segments · Separator (`powerline` `powerline-thin` `slash` `pipe` `ascii`) · Symbols (`nerdfont` `unicode` `ascii` — stored in settings.json as `nerd`/`unicode`/`ascii`) · Border style (`rounded` `heavy` `double` `single`) · Rainbow border on max thinking · Animate rainbow border |
 | Top Left Segment Group | Pi symbol · Model · Provider · Thinking level · Path · Git · PR |
 | Top Right Segment Group | Token rate · Session name |
 | Bottom Right Segment Group | Feeds · Token rate · Pi stats · Context bar · Context stats |
 | Bottom Left Segment Group | Scroll hint · Feeds · Token rate |
 | Feeds | One subscription per row: type · field · prefix · format, plus add/remove |
-| Defaults | Transparent on · Separator `powerline-thin` · Symbols `nerdfont` · Border style `rounded` · Rainbow border on |
+| Defaults | Transparent on · Separator `powerline-thin` · Symbols `nerdfont` · Border style `rounded` · Rainbow border on · Animate rainbow border on |
 
 With **Rainbow border on max thinking** on (the default), cycling the thinking level to `max`
-replaces the border's fixed theme color with an animated rainbow: a full hue cycle distributed
-around the box perimeter that slowly flows around the border (~14s per rotation), like Apple
-Intelligence's screen border. Any other thinking level keeps the normal theme border color.
-The settings preview animates while the toggle is on; the effect repaints the border about
-20 times a second, so switch it off over slow SSH links or in terminals with expensive
-redraws.
+replaces the border's fixed theme color with a rainbow: a full hue cycle distributed around the
+box perimeter. **Animate rainbow border** is also on by default, so the hues flow around the
+border (~14s per rotation), like Apple Intelligence's screen border. Turn animation off to keep
+the rainbow at a fixed color phase without its repaint timer; the settings preview uses a stable
+phase too. Any other thinking level keeps the normal theme border color. Disable animation over
+slow SSH links or in terminals with expensive redraws while retaining the rainbow border.
 
 ## Segments
 
@@ -130,9 +131,10 @@ A feed contributes nothing when its publisher is absent, has published nothing t
 reports a value its format suppresses; the segment disappears entirely once every feed is
 quiet, so it costs nothing to leave switched on. Publishers typically reset per process run,
 so entries predating the current session start (`--resume`, `/reload`, `/new`, fork) are
-ignored rather than shown as a stale total. The session is re-read at most once every two
-seconds, because pi copies the whole entry list on each read while the bar re-renders on
-every keystroke.
+ignored rather than shown as a stale total. During steady-state rendering, the session is
+re-read at most once every two seconds, because pi copies the whole entry list on each read
+while the bar re-renders on every keystroke; changing subscriptions or starting a new
+session forces an immediate scan.
 
 ### Publishing a feed
 

package/index.ts

@@ -55,17 +55,18 @@ export default function (pi: ExtensionAPI) {
 	rateMonitor.attach(pi);
 	builder.setTokenRateProvider(() => rateMonitor.getDisplay());
 
-	// Animated rainbow border while the thinking level is max: a timer steps
-	// the hue phase and re-renders; renderBoxed paints the box glyphs from it.
+	// The rainbow is visible at max thinking independently of whether its hue
+	// phase is animated. Stopping the timer preserves the current live phase.
 	const rainbow = new RainbowBorder();
 	let rainbowTimer: ReturnType<typeof setInterval> | undefined;
 	const rainbowActive = (): boolean =>
 		state.effective.rainbowBorder && activeCtx !== undefined && pi.getThinkingLevel() === "max";
+	const rainbowAnimationActive = (): boolean => rainbowActive() && state.effective.rainbowAnimation;
 	const syncRainbow = (): void => {
-		if (rainbowActive()) {
+		if (rainbowAnimationActive()) {
 			if (!rainbowTimer) {
 				rainbowTimer = setInterval(() => {
-					if (!rainbowActive()) {
+					if (!rainbowAnimationActive()) {
 						syncRainbow();
 						return;
 					}

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@underactive/pi-topping-statusline",
-	"version": "0.1.2",
+	"version": "0.1.3",
 	"description": "oh-my-pi's powerline statusline as a pi extension with some customizations",
 	"keywords": [
 		"pi-package",

package/src/context.ts

@@ -191,6 +191,9 @@ export class SegmentContextBuilder {
 			this.#requestRender();
 		} finally {
 			this.#repoResolving = false;
+			if (this.#repoCwd && this.#repoCwd !== cwd) {
+				void this.#resolveRepo(this.#repoCwd);
+			}
 		}
 	}
 
@@ -278,7 +281,8 @@ export class SegmentContextBuilder {
 			// first: matching on customType alone lets one shadow a real entry.
 			if (entry?.type !== "custom" || !pending.has(entry.customType)) continue;
 			const ts = Date.parse(entry.timestamp);
-			if (!Number.isFinite(ts) || ts < this.#runStartedAt) break;
+			if (!Number.isFinite(ts)) continue;
+			if (ts < this.#runStartedAt) break;
 			pending.delete(entry.customType);
 			if (entry.data !== undefined) this.#feedData[entry.customType] = entry.data;
 		}
@@ -307,7 +311,7 @@ export class SegmentContextBuilder {
 						const u = URL.parse(parsed.url);
 						if (
 							u &&
-							(u.protocol === "https:" || u.protocol === "http:") &&
+							u.protocol === "https:" &&
 							u.hostname === "github.com" &&
 							!/[\u0000-\u001f\u007f]/.test(parsed.url)
 						) {

package/src/settings-menu.ts

@@ -135,6 +135,11 @@ function buildSections(settings: StatusLineSettings): MenuSection[] {
 				label: "Rainbow border on max thinking",
 				value: settings.rainbowBorder ?? true,
 			},
+			{
+				id: "rainbowAnimation",
+				label: "Animate rainbow border",
+				value: settings.rainbowAnimation ?? true,
+			},
 		],
 	};
 	return [
@@ -164,6 +169,7 @@ function valuesToSettings(values: Record<string, MenuValue>): StatusLineSettings
 		segments,
 		feeds: sanitizeFeeds(feedsFromValues(values)),
 		rainbowBorder: values.rainbowBorder === true,
+		rainbowAnimation: values.rainbowAnimation === true,
 	};
 }
 
@@ -195,12 +201,10 @@ function cannedFeedData(
 }
 
 class StatusLinePreview {
-	readonly #state: SettingsState;
 	readonly #builder: SegmentContextBuilder;
 	readonly #uiTheme: Theme;
 
-	constructor(state: SettingsState, builder: SegmentContextBuilder, uiTheme: Theme) {
-		this.#state = state;
+	constructor(builder: SegmentContextBuilder, uiTheme: Theme) {
 		this.#builder = builder;
 		this.#uiTheme = uiTheme;
 	}
@@ -217,6 +221,7 @@ class StatusLinePreview {
 		const boxWidth = Math.max(24, innerWidth - 1);
 		const barWidth = boxWidth - 6;
 		const rainbowOn = effective.rainbowBorder;
+		const rainbowAnimationOn = rainbowOn && effective.rainbowAnimation;
 
 		// The theme singleton drives glyph lookups everywhere, so swap the
 		// symbol preset for this synchronous render only; the live bar behind
@@ -255,7 +260,7 @@ class StatusLinePreview {
 			const bottomIdx = 2;
 			const painters = makeBoxPainters({
 				rainbowOn,
-				rainbow: new RainbowBorder((elapsedMs * 360) / RAINBOW_CYCLE_MS),
+				rainbow: new RainbowBorder(rainbowAnimationOn ? (elapsedMs * 360) / RAINBOW_CYCLE_MS : 0),
 				box,
 				width: boxWidth,
 				bottomIdx,
@@ -291,9 +296,8 @@ class StatusLinePreview {
 						painters.paint(1, boxWidth - 1, box.vertical),
 					renderBoxRow(painters, bottom, bottomIdx, boxWidth, box.bottomLeft, box.bottomRight),
 				],
-				// Keep the preview animated only while the rainbow is enabled;
-				// a static preview refreshes once and stops.
-				nextRefreshInMs: rainbowOn ? RAINBOW_FRAME_MS : undefined,
+				// A static rainbow stays at phase zero and needs no repaint timer.
+				nextRefreshInMs: rainbowAnimationOn ? RAINBOW_FRAME_MS : undefined,
 			};
 		} finally {
 			theme.setSymbolPreset(applied);
@@ -308,13 +312,13 @@ export function registerSettingsCommand(
 	onChange: () => void,
 ): void {
 	pi.registerCommand("topping-statusline-settings", {
-		description: "Configure transparent segments, statusline segments, separator, symbols, border style, feeds, and rainbow border",
+		description: "Configure transparent segments, statusline segments, separator, symbols, border style, feeds, and rainbow border animation",
 		handler: async (_args, ctx: ExtensionCommandContext) => {
 			if (ctx.mode !== "tui") {
 				ctx.ui.notify("/topping-statusline-settings requires TUI mode", "error");
 				return;
 			}
-			const preview = new StatusLinePreview(state, builder, ctx.ui.theme);
+			const preview = new StatusLinePreview(builder, ctx.ui.theme);
 			const result = await showMenu<Record<string, MenuValue>>(ctx, {
 				title: "Pi Topping Statusline: Settings",
 				sections: buildSections(state.settings),

package/src/settings.ts

@@ -92,6 +92,7 @@ function loadSettings(): StatusLineSettings {
 		}
 		if (raw.feeds !== undefined) settings.feeds = sanitizeFeeds(raw.feeds);
 		if (typeof raw.rainbowBorder === "boolean") settings.rainbowBorder = raw.rainbowBorder;
+		if (typeof raw.rainbowAnimation === "boolean") settings.rainbowAnimation = raw.rainbowAnimation;
 		return settings;
 	} catch {
 		// Missing or corrupt settings.json falls back to defaults.
@@ -153,6 +154,7 @@ export function resolveEffectiveSettings(settings: StatusLineSettings): Effectiv
 		bottomLeftSegments,
 		bottomRightSegments,
 		rainbowBorder: settings.rainbowBorder ?? true,
+		rainbowAnimation: settings.rainbowAnimation ?? true,
 		includes,
 		segmentOptions: {
 			model: { showModel: seg.model, showProvider: seg.provider, showThinking: seg.thinking },

package/src/tests/rainbow.test.ts

@@ -113,9 +113,11 @@ test("a left group shifts the gap start by its width", () => {
 	assert.ok(calls[0]!.startCol > 3, "gap starts after the pi segment, not at the origin");
 });
 
-test("rainbowBorder defaults to true and is loadable", () => {
+test("rainbow settings default to true and resolve explicit false", () => {
 	assert.equal(resolveEffectiveSettings({}).rainbowBorder, true);
+	assert.equal(resolveEffectiveSettings({}).rainbowAnimation, true);
 	assert.equal(resolveEffectiveSettings({ rainbowBorder: false }).rainbowBorder, false);
+	assert.equal(resolveEffectiveSettings({ rainbowAnimation: false }).rainbowAnimation, false);
 });
 
 test("borderStyle defaults to rounded and is loadable", () => {

package/src/types.ts

@@ -69,8 +69,10 @@ export interface StatusLineSettings {
 	borderStyle?: BorderStyle;
 	segments?: StatusLineSegmentToggles;
 	feeds?: StatusLineFeed[];
-	/** Animated rainbow border while the thinking level is max. */
+	/** Rainbow border while the thinking level is max. */
 	rainbowBorder?: boolean;
+	/** Animate the rainbow border when it is visible. */
+	rainbowAnimation?: boolean;
 }
 
 /** Resolved per-segment render options; resolveEffectiveSettings always fills them. */
@@ -102,6 +104,7 @@ export interface EffectiveStatusLineSettings {
 	bottomRightSegments: StatusLineSegmentId[];
 	segmentOptions: StatusLineSegmentOptions;
 	rainbowBorder: boolean;
+	rainbowAnimation: boolean;
 	includes: SegmentIncludes;
 }