@underactive/pi-topping 0.7.2 → 0.8.0

package/README.md

@@ -24,7 +24,7 @@ Restart Pi (or run `/reload`) to pick it up.
 
 ### Sibling toppings
 
-The sibling-topping setup flow currently ships behind the `PI_TOPPING_SIBLING_SETUP=1` environment flag. When enabled, run `/topping-setup` in TUI mode to selectively install missing Pi Topping sibling extensions or uninstall sibling packages recorded in Pi's user settings. Installation rows are selected by default; removal rows are unchecked by default. Restart Pi after applying package changes. Use `/topping-setup disable-side-toppings-check` to suppress missing-topping notices, or `enable-side-toppings-check` to restore them.
+Run `/topping-setup` in TUI mode to selectively install missing Pi Topping sibling extensions or uninstall sibling packages recorded in Pi's user settings. Installation rows are selected by default; removal rows are unchecked by default. Restart Pi after applying package changes. Use `/topping-setup disable-side-toppings-check` to suppress missing-topping notices, or `enable-side-toppings-check` to restore them.
 
 ## Settings
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@underactive/pi-topping",
-  "version": "0.7.2",
+  "version": "0.8.0",
   "description": "A Pi extension that adds quality of life improvements to Pi's UX: a high-vis user prompt, a shimmering working loader with token activity meter and elapsed timer, and an end-of-turn completion marker",
   "keywords": ["pi-package", "pi-extension"],
   "license": "MIT",

package/src/session.ts

@@ -38,7 +38,6 @@ import {
 	StreamingWordCounter,
 	type ThinkingLevel,
 } from "./format.ts";
-import { isSiblingSetupEnabled } from "./flags.ts";
 import { showMenu } from "./menu.ts";
 import { getResponseModelColorizer } from "./nvidia-green.ts";
 import { registerSetupCommand } from "./setup-command.ts";
@@ -181,7 +180,7 @@ export class SessionManager {
 		this.#state.activityMeter.setDirection(this.#settings.decorations.meterDirection);
 		if (this.usable(ctx)) {
 			this.applyIndicator(ctx);
-			if (isSiblingSetupEnabled()) announceMissingToppingsOnce(this.#pi);
+			announceMissingToppingsOnce(this.#pi);
 		}
 	};
 
@@ -395,13 +394,13 @@ export class SessionManager {
 		this.#pi.on("ui_prompt_end", this.#onUIPromptEnd);
 		this.#pi.on("session_shutdown", this.#onSessionShutdown);
 		this.#pi.registerEntryRenderer<DoneEntryData>(DONE_ENTRY_TYPE, (entry, _o, theme) => this.#renderDoneEntry(entry, theme));
-		if (isSiblingSetupEnabled()) this.#pi.registerEntryRenderer<MissingToppingsEntryData>(SETUP_ENTRY_TYPE, (entry, _o, theme) => renderMissingToppingsEntry(entry, theme));
+		this.#pi.registerEntryRenderer<MissingToppingsEntryData>(SETUP_ENTRY_TYPE, (entry, _o, theme) => renderMissingToppingsEntry(entry, theme));
 		this.#pi.registerMessageRenderer<PromptBoxDetails>(PROMPT_BOX_TYPE, promptBoxRenderer);
 		this.#pi.registerCommand("topping-settings", {
 			description: "Configure prompt decoration, working-loader features/order, and completion-marker settings.",
 			handler: async (_a, ctx) => this.showSettings(ctx),
 		});
-		if (isSiblingSetupEnabled()) registerSetupCommand(this.#pi);
+		registerSetupCommand(this.#pi);
 	}
 
 	private usable(ctx: ExtensionContext): boolean {

package/src/flags.ts

@@ -1,19 +0,0 @@
-/**
- * Feature gates for unreleased functionality.
- *
- * Sibling-topping detection and the `/topping-setup` command ship dark: the
- * default below stays `false` until the flow is ready for the public. Setting
- * `PI_TOPPING_SIBLING_SETUP` to any value other than `0`, `false`, `off`, or
- * `no` (case-insensitive) opts a session in without touching code.
- */
-
-const SIBLING_SETUP_DEFAULT = false;
-
-const FALSEY_FLAG_VALUES = new Set(["0", "false", "off", "no"]);
-
-/** Whether the sibling-topping setup flow is registered and advertised. */
-export function isSiblingSetupEnabled(): boolean {
-	const value = process.env.PI_TOPPING_SIBLING_SETUP?.trim().toLowerCase();
-	if (value === undefined || value === "") return SIBLING_SETUP_DEFAULT;
-	return !FALSEY_FLAG_VALUES.has(value);
-}