@omg-dev/vite-plugin 0.4.30 → 0.4.31

package/dist/index.mjs

@@ -3,7 +3,7 @@ import fs from "node:fs";
 import { schemaToCollections, schemaToDrizzle, schemaToZod } from "@omg-dev/schema";
 import { scanTriggers } from "@omg-dev/server/trigger-scan";
 //#region src/auth-bridge.ts
-const AUTH_RELAY_SKIP_HEADERS = new Set([
+const AUTH_RELAY_SKIP_HEADERS = /* @__PURE__ */ new Set([
 	"set-cookie",
 	"content-encoding",
 	"content-length",
@@ -48,7 +48,7 @@ async function fetchAuthUpstream(kind, init, qs = "") {
 }
 //#endregion
 //#region src/scanner.ts
-const HTTP_METHODS = new Set([
+const HTTP_METHODS = /* @__PURE__ */ new Set([
 	"GET",
 	"POST",
 	"PUT",
@@ -180,6 +180,8 @@ function extractExports(source) {
 const EXPORT_WORKFLOW_RE = /^export\s+const\s+(\w+)\s*(?::\s*[^=]+)?\s*=\s*workflow\s*\(\s*(.+?)\s*,/gm;
 const NAME_RE = /^[A-Za-z][A-Za-z0-9_]*$/;
 var WorkflowScanError = class extends Error {
+	file;
+	detail;
 	constructor(file, detail) {
 		super(`[vibes:workflows] ${file}: ${detail}`);
 		this.file = file;
@@ -464,8 +466,6 @@ function vibes(opts = {}) {
 	const pwaEnabled = opts.pwa !== false;
 	const pwaOpts = typeof opts.pwa === "object" ? opts.pwa : {};
 	let pwaAutoInjected = false;
-	const feedbackEnabled = opts.feedback !== false;
-	let feedbackInjected = false;
 	const brandBadgeEnabled = opts.brandBadge !== false;
 	let brandBadgeInjected = false;
 	function existingIconFiles() {
@@ -484,10 +484,7 @@ function vibes(opts = {}) {
 		name: "vibes",
 		enforce: "pre",
 		config() {
-			return {
-				define: { __VIBES_BRAND_FEEDBACK__: JSON.stringify(brandBadgeEnabled && feedbackEnabled) },
-				server: { watch: { ignored: ["**/.vibes/**"] } }
-			};
+			return { server: { watch: { ignored: ["**/.vibes/**"] } } };
 		},
 		configResolved(resolvedConfig) {
 			config = resolvedConfig;
@@ -771,7 +768,8 @@ window.__VIBES_AUTH_SESSION_URL = "/__vibes/auth/session";
 					if (bearerProto) {
 						const token = bearerProto.slice(13);
 						try {
-							userId = (await subAuthMW(new Request("http://localhost/__vibes_sub", { headers: { authorization: `Bearer ${token}` } })))?.userId ?? null;
+							const fakeReq = new Request("http://localhost/__vibes_sub", { headers: { authorization: `Bearer ${token}` } });
+							userId = (await subAuthMW(fakeReq))?.userId ?? null;
 						} catch {}
 					}
 					wss.handleUpgrade(req, socket, head, (ws) => {
@@ -786,7 +784,8 @@ window.__VIBES_AUTH_SESSION_URL = "/__vibes/auth/session";
 						};
 						addSubClient(subClient);
 						ws.on("message", (data) => {
-							handleSubMessage(subClient, typeof data === "string" ? data : Buffer.isBuffer(data) ? data.toString("utf8") : Array.isArray(data) ? Buffer.concat(data).toString("utf8") : Buffer.from(data).toString("utf8"));
+							const raw = typeof data === "string" ? data : Buffer.isBuffer(data) ? data.toString("utf8") : Array.isArray(data) ? Buffer.concat(data).toString("utf8") : Buffer.from(data).toString("utf8");
+							handleSubMessage(subClient, raw);
 						});
 						const closeOut = () => {
 							if (subClient.readyState === 3) return;
@@ -990,7 +989,7 @@ window.__VIBES_AUTH_SESSION_URL = "/__vibes/auth/session";
 		},
 		async transform(code, id) {
 			if (config?.command !== "build") return null;
-			if (!pwaEnabled && !feedbackEnabled && !brandBadgeEnabled) return null;
+			if (!pwaEnabled && !brandBadgeEnabled) return null;
 			if (!isAppEntry(id, resolvedRoot)) return null;
 			const imports = [];
 			if (brandBadgeEnabled && !brandBadgeInjected) if (!await this.resolve("@omg-dev/sdk/brand/auto", id)) this.warn("[vibes] brand badge: @omg-dev/sdk/brand/auto is not available — badge skipped. Update @omg-dev/sdk, or silence this with vibes({ brandBadge: false }).");
@@ -998,11 +997,6 @@ window.__VIBES_AUTH_SESSION_URL = "/__vibes/auth/session";
 				brandBadgeInjected = true;
 				imports.push(`import "@omg-dev/sdk/brand/auto";`);
 			}
-			if (feedbackEnabled && !brandBadgeEnabled && !feedbackInjected) if (!await this.resolve("@omg-dev/sdk/feedback/auto", id)) this.warn("[vibes] feedback: @omg-dev/sdk is not installed — shake-to-report skipped. Add @omg-dev/sdk to dependencies, or silence this with vibes({ feedback: false }).");
-			else {
-				feedbackInjected = true;
-				imports.push(`import "@omg-dev/sdk/feedback/auto";`);
-			}
 			if (pwaEnabled && pwaOpts.autoPrompt !== false && !pwaAutoInjected) if (!await this.resolve("@omg-dev/pwa/auto", id)) this.warn("[vibes] pwa: @omg-dev/pwa is not installed — soft install prompt skipped. Add @omg-dev/pwa to dependencies, or silence this with vibes({ pwa: { autoPrompt: false } }).");
 			else {
 				pwaAutoInjected = true;
@@ -1018,7 +1012,8 @@ window.__VIBES_AUTH_SESSION_URL = "/__vibes/auth/session";
 			if (!pwaEnabled) return;
 			if (userManifestHref()) return;
 			const indexPath = path.join(resolvedRoot, "index.html");
-			const cfg = resolvePwaConfig(pwaOpts, fs.existsSync(indexPath) ? fs.readFileSync(indexPath, "utf-8") : "");
+			const indexHtml = fs.existsSync(indexPath) ? fs.readFileSync(indexPath, "utf-8") : "";
+			const cfg = resolvePwaConfig(pwaOpts, indexHtml);
 			this.emitFile({
 				type: "asset",
 				fileName: MANIFEST_FILE,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@omg-dev/vite-plugin",
-  "version": "0.4.30",
+  "version": "0.4.31",
   "type": "module",
   "exports": {
     ".": {
@@ -15,8 +15,8 @@
     "build": "vp pack src/index.ts --no-fail-on-warn"
   },
   "dependencies": {
-    "@omg-dev/server": "0.4.30",
-    "@omg-dev/schema": "0.4.30",
+    "@omg-dev/server": "0.4.31",
+    "@omg-dev/schema": "0.4.31",
     "ws": "^8.18.0"
   },
   "devDependencies": {

package/src/brand-badge-inject.test.ts

@@ -29,7 +29,7 @@ async function runTransform(
 }
 
 function buildPlugin(opts?: Parameters<typeof vibes>[0]) {
-  const plugin = vibes({ root: ROOT, pwa: false, feedback: false, ...opts })
+  const plugin = vibes({ root: ROOT, pwa: false, ...opts })
   const configResolved = plugin.configResolved as (cfg: unknown) => void
   configResolved({ command: "build", root: ROOT })
   return plugin
@@ -71,17 +71,6 @@ describe("brand badge injection", () => {
     expect(context.warnings.some((w) => w.includes("brand badge"))).toBe(true)
   })
 
-  test("badge ON bundles feedback: no standalone feedback import injected", async () => {
-    // brand badge default ON, with feedback + pwa also on. The badge carries
-    // feedback (button-less), so standalone feedback/auto must not be appended.
-    const plugin = vibes({ root: ROOT })
-    const configResolved = plugin.configResolved as (cfg: unknown) => void
-    configResolved({ command: "build", root: ROOT })
-    const out = await runTransform(plugin, "export default 1", ENTRY, ctx())
-    expect(out?.code).toContain(BRAND_IMPORT)
-    expect(out?.code).not.toContain(`import "@omg-dev/sdk/feedback/auto";`)
-  })
-
   test("serve mode never injects the badge", async () => {
     const plugin = vibes({ root: ROOT })
     const configResolved = plugin.configResolved as (cfg: unknown) => void

package/src/index.ts

@@ -34,12 +34,6 @@ export interface VibesOptions {
    * install prompt. Default ON; pass `false` to opt out, or an options
    * object to customize (see PwaOptions). */
   pwa?: boolean | PwaOptions
-  /** Shake-to-report feedback widget for published builds — auto-mounts
-   * <VibesFeedback/> outside the app's React tree (shake / two-finger press
-   * summons it; report → agent /_report → LFG triage). Default ON; pass
-   * `false` to opt out and place <VibesFeedback/> from @omg-dev/sdk/feedback
-   * manually. */
-  feedback?: boolean
   /** Small "omg" badge for published apps. Auto-mounts outside the app's React
    * tree and opens a built-in "What is omg?" dialog. Default ON; pass `false`
    * to omit it for a specific app while this rolls out. */
@@ -57,8 +51,6 @@ export default function vibes(opts: VibesOptions = {}): Plugin {
   const pwaOpts: PwaOptions = typeof opts.pwa === "object" ? opts.pwa : {}
   let pwaAutoInjected = false
 
-  const feedbackEnabled = opts.feedback !== false
-  let feedbackInjected = false
   const brandBadgeEnabled = opts.brandBadge !== false
   let brandBadgeInjected = false
 
@@ -95,13 +87,6 @@ export default function vibes(opts: VibesOptions = {}): Plugin {
     // browser, so blanket-ignore it.
     config() {
       return {
-        // When the brand badge is on (default), it bundles the feedback sheet
-        // (no separate floating button). Expose whether feedback is actually
-        // enabled so the badge can omit it when the app opts out via
-        // vibes({ feedback: false }).
-        define: {
-          __VIBES_BRAND_FEEDBACK__: JSON.stringify(brandBadgeEnabled && feedbackEnabled),
-        },
         server: {
           watch: {
             ignored: ["**/.vibes/**"],
@@ -777,7 +762,7 @@ window.__VIBES_AUTH_SESSION_URL = "/__vibes/auth/session";
     // of main.tsx and never load in dev preview.
     async transform(code: string, id: string) {
       if (config?.command !== "build") return null
-      if (!pwaEnabled && !feedbackEnabled && !brandBadgeEnabled) return null
+      if (!pwaEnabled && !brandBadgeEnabled) return null
       if (!isAppEntry(id, resolvedRoot)) return null
 
       const imports: string[] = []
@@ -798,28 +783,6 @@ window.__VIBES_AUTH_SESSION_URL = "/__vibes/auth/session";
         }
       }
 
-      // Shake-to-report feedback widget — opt-out via vibes({ feedback: false }).
-      // Build-only + outside the React tree, so it survives agent rewrites of
-      // main.tsx and never arms a gesture listener in the dev preview iframe.
-      //
-      // When the brand badge is on (default) it ALREADY mounts a button-less
-      // feedback sheet and summons it from its dialog, so injecting the
-      // standalone widget here would double-mount it (two floating controls —
-      // the exact duplication we're collapsing). Only inject standalone feedback
-      // when the brand badge is disabled.
-      if (feedbackEnabled && !brandBadgeEnabled && !feedbackInjected) {
-        const resolvedFb = await this.resolve("@omg-dev/sdk/feedback/auto", id)
-        if (!resolvedFb) {
-          this.warn(
-            "[vibes] feedback: @omg-dev/sdk is not installed — shake-to-report skipped. " +
-              "Add @omg-dev/sdk to dependencies, or silence this with vibes({ feedback: false }).",
-          )
-        } else {
-          feedbackInjected = true
-          imports.push(`import "@omg-dev/sdk/feedback/auto";`)
-        }
-      }
-
       // Soft install prompt — opt-out via vibes({ pwa: { autoPrompt: false } }).
       if (pwaEnabled && pwaOpts.autoPrompt !== false && !pwaAutoInjected) {
         const resolved = await this.resolve("@omg-dev/pwa/auto", id)

package/src/feedback-inject.test.ts

@@ -1,79 +0,0 @@
-// Verifies the build-time injection of the shake-to-report feedback widget
-// (@omg-dev/sdk/feedback/auto) into the app entry. The brand badge bundles
-// feedback when it's on (the default), so the STANDALONE widget only injects
-// when the brand badge is disabled — these tests run with brandBadge:false.
-// Opt-out via vibes({ feedback: false }), build-only, app-entry-only.
-
-import { describe, expect, test } from "bun:test"
-import vibes from "./index.ts"
-
-const ROOT = "/app"
-const ENTRY = `${ROOT}/src/main.tsx`
-const FEEDBACK_IMPORT = `import "@omg-dev/sdk/feedback/auto";`
-
-// Minimal Rollup plugin-context stub: resolve() succeeds for everything,
-// warn() is captured. transform() is a plain async fn we bind `this` onto.
-function ctx() {
-  const warnings: string[] = []
-  return {
-    warnings,
-    resolve: async (src: string) => ({ id: src }),
-    warn: (msg: string) => warnings.push(msg),
-  }
-}
-
-async function runTransform(
-  plugin: ReturnType<typeof vibes>,
-  code: string,
-  id: string,
-  context: ReturnType<typeof ctx>,
-) {
-  const fn = plugin.transform as (this: unknown, code: string, id: string) => Promise<unknown>
-  return fn.call(context, code, id) as Promise<{ code: string } | null>
-}
-
-function buildPlugin(opts?: Parameters<typeof vibes>[0]) {
-  const plugin = vibes({ root: ROOT, brandBadge: false, ...opts })
-  // configResolved sets command + root that transform gates on.
-  const configResolved = plugin.configResolved as (cfg: unknown) => void
-  configResolved({ command: "build", root: ROOT })
-  return plugin
-}
-
-describe("feedback widget injection", () => {
-  test("default ON: appends the auto-mount import to the app entry", async () => {
-    const plugin = buildPlugin()
-    const out = await runTransform(plugin, "export default 1", ENTRY, ctx())
-    expect(out?.code).toContain(FEEDBACK_IMPORT)
-  })
-
-  test("opt-out via feedback:false omits the import", async () => {
-    const plugin = buildPlugin({ feedback: false, pwa: false })
-    const out = await runTransform(plugin, "export default 1", ENTRY, ctx())
-    // Nothing to inject → transform returns null.
-    expect(out).toBeNull()
-  })
-
-  test("injected exactly once across multiple entry transforms", async () => {
-    const plugin = buildPlugin({ pwa: false })
-    const first = await runTransform(plugin, "export default 1", ENTRY, ctx())
-    const second = await runTransform(plugin, "export default 2", ENTRY, ctx())
-    expect(first?.code).toContain(FEEDBACK_IMPORT)
-    expect(second).toBeNull() // already injected
-  })
-
-  test("non-entry modules are untouched", async () => {
-    const plugin = buildPlugin()
-    const out = await runTransform(plugin, "export const x = 1", `${ROOT}/src/util.ts`, ctx())
-    expect(out).toBeNull()
-  })
-
-  test("warns (and skips) when @omg-dev/sdk cannot be resolved", async () => {
-    const plugin = buildPlugin({ pwa: false })
-    const context = ctx()
-    context.resolve = async () => null // simulate missing dependency
-    const out = await runTransform(plugin, "export default 1", ENTRY, context)
-    expect(out).toBeNull()
-    expect(context.warnings.some((w) => w.includes("shake-to-report skipped"))).toBe(true)
-  })
-})