@omg-dev/vite-plugin 0.4.29 → 0.4.30

package/dist/index.mjs

@@ -457,29 +457,6 @@ function isAppEntry(id, root) {
 }
 //#endregion
 //#region src/index.ts
-async function pwaDeclaresSubpathExport(resolve, importer, subpath) {
-	try {
-		const rootEntry = await resolve("@omg-dev/pwa", importer);
-		if (!rootEntry?.id) return false;
-		let dir = path.dirname(rootEntry.id.split("?")[0]);
-		for (let i = 0; i < 12; i++) {
-			const pkgPath = path.join(dir, "package.json");
-			if (fs.existsSync(pkgPath)) try {
-				const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf-8"));
-				if (pkg.name === "@omg-dev/pwa") {
-					const exp = pkg.exports;
-					return !!exp && typeof exp === "object" && subpath in exp;
-				}
-			} catch {}
-			const parent = path.dirname(dir);
-			if (parent === dir) break;
-			dir = parent;
-		}
-		return false;
-	} catch {
-		return false;
-	}
-}
 function vibes(opts = {}) {
 	let resolvedRoot;
 	let config;
@@ -487,7 +464,6 @@ function vibes(opts = {}) {
 	const pwaEnabled = opts.pwa !== false;
 	const pwaOpts = typeof opts.pwa === "object" ? opts.pwa : {};
 	let pwaAutoInjected = false;
-	let remixCtaInjected = false;
 	const feedbackEnabled = opts.feedback !== false;
 	let feedbackInjected = false;
 	const brandBadgeEnabled = opts.brandBadge !== false;
@@ -1032,12 +1008,6 @@ window.__VIBES_AUTH_SESSION_URL = "/__vibes/auth/session";
 				pwaAutoInjected = true;
 				imports.push(`import "@omg-dev/pwa/auto";`);
 			}
-			if (pwaEnabled && !brandBadgeEnabled && !remixCtaInjected) {
-				if (await pwaDeclaresSubpathExport((source, importer) => this.resolve(source, importer), id, "./remix-cta")) {
-					remixCtaInjected = true;
-					imports.push(`import "@omg-dev/pwa/remix-cta";`);
-				}
-			}
 			if (imports.length === 0) return null;
 			return {
 				code: `${code}\n${imports.join("\n")}\n`,

package/package.json

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

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

@@ -71,17 +71,15 @@ describe("brand badge injection", () => {
     expect(context.warnings.some((w) => w.includes("brand badge"))).toBe(true)
   })
 
-  test("badge ON bundles feedback + remix: no standalone imports injected", async () => {
+  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) and the "Make it mine" CTA itself, so the
-    // standalone feedback/auto + remix-cta imports must NOT be appended.
+    // 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";`)
-    expect(out?.code).not.toContain(`import "@omg-dev/pwa/remix-cta";`)
   })
 
   test("serve mode never injects the badge", async () => {

package/src/index.ts

@@ -46,47 +46,6 @@ export interface VibesOptions {
   brandBadge?: boolean
 }
 
-// Resolve @omg-dev/pwa for `importer` and report whether its package.json declares
-// the given subpath (e.g. "./remix-cta") in its `exports` map. Used to keep the
-// build-time CTA injection skew-safe: an older baked @omg-dev/pwa that predates a
-// subpath export must NOT get the import appended, or the bundle fails to resolve.
-// Returns false on any resolution / parse hiccup — the CTA is additive, so when
-// in doubt we skip it rather than risk breaking the user's build.
-async function pwaDeclaresSubpathExport(
-  resolve: (source: string, importer: string) => Promise<{ id: string } | null>,
-  importer: string,
-  subpath: string,
-): Promise<boolean> {
-  try {
-    // Resolve the always-present root export, then walk up to the owning
-    // package.json. We can't resolve "@omg-dev/pwa/package.json" directly because
-    // an older exports map may not expose "./package.json".
-    const rootEntry = await resolve("@omg-dev/pwa", importer)
-    if (!rootEntry?.id) return false
-    let dir = path.dirname(rootEntry.id.split("?")[0])
-    for (let i = 0; i < 12; i++) {
-      const pkgPath = path.join(dir, "package.json")
-      if (fs.existsSync(pkgPath)) {
-        try {
-          const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf-8"))
-          if (pkg.name === "@omg-dev/pwa") {
-            const exp = pkg.exports
-            return !!exp && typeof exp === "object" && subpath in exp
-          }
-        } catch {
-          // Unreadable/!JSON package.json — keep climbing toward the real one.
-        }
-      }
-      const parent = path.dirname(dir)
-      if (parent === dir) break
-      dir = parent
-    }
-    return false
-  } catch {
-    return false
-  }
-}
-
 // ── Plugin ────────────────────────────────────────────────────────────────────
 
 export default function vibes(opts: VibesOptions = {}): Plugin {
@@ -97,7 +56,6 @@ export default function vibes(opts: VibesOptions = {}): Plugin {
   const pwaEnabled = opts.pwa !== false
   const pwaOpts: PwaOptions = typeof opts.pwa === "object" ? opts.pwa : {}
   let pwaAutoInjected = false
-  let remixCtaInjected = false
 
   const feedbackEnabled = opts.feedback !== false
   let feedbackInjected = false
@@ -813,11 +771,8 @@ window.__VIBES_AUTH_SESSION_URL = "/__vibes/auth/session";
     },
 
     // ── transform ─────────────────────────────────────────────────────────────
-    // Published builds get two side-effect imports appended to the app entry:
+    // Published builds get platform side-effect imports appended to the app entry:
     //   - @omg-dev/pwa/auto       — the soft install prompt (gated by autoPrompt)
-    //   - @omg-dev/pwa/remix-cta  — the "Make your own" remix bridge (always on
-    //                             when pwa is enabled; self-gates on the baked
-    //                             VITE_APP_SLUG, so it's inert if not published)
     // Build-time injection (not template source) so both survive agent rewrites
     // of main.tsx and never load in dev preview.
     async transform(code: string, id: string) {
@@ -879,33 +834,6 @@ window.__VIBES_AUTH_SESSION_URL = "/__vibes/auth/session";
         }
       }
 
-      // "Make your own" remix CTA — independent of autoPrompt. It self-gates at
-      // runtime on import.meta.env.VITE_APP_SLUG (baked by the orchestrator on
-      // deploy), so it renders nothing on a non-published build.
-      //
-      // Skew guard: the remix-cta subpath shipped in @omg-dev/pwa >= 0.4.16. An app
-      // whose baked @omg-dev/pwa predates it (e.g. 0.4.12) has no "./remix-cta" in
-      // its exports map, so injecting the import hard-fails the build with
-      // "Errored while resolving @omg-dev/pwa/remix-cta". this.resolve() alone is not
-      // a reliable gate across this version skew, so only inject once we've
-      // confirmed the resolved package actually declares the subpath export.
-      //
-      // The brand badge (default on) now hosts "Make it mine" itself: same omg
-      // mark, remix CTA first, and it collapses to the plain branding badge on
-      // dismiss. So only inject the standalone remix-cta when the brand badge is
-      // off — otherwise we'd render two corner pills with the same intent.
-      if (pwaEnabled && !brandBadgeEnabled && !remixCtaInjected) {
-        const exported = await pwaDeclaresSubpathExport(
-          (source, importer) => this.resolve(source, importer),
-          id,
-          "./remix-cta",
-        )
-        if (exported) {
-          remixCtaInjected = true
-          imports.push(`import "@omg-dev/pwa/remix-cta";`)
-        }
-      }
-
       if (imports.length === 0) return null
       return { code: `${code}\n${imports.join("\n")}\n`, map: null }
     },