#8227: error if showing temporary panel from frame (#8231)

Co-authored-by: Eduardo Fungairino <[email protected]>

src/bricks/transformers/temporaryInfo/DisplayTemporaryInfo.test.ts

@@ -55,16 +55,21 @@ import { contextAsPlainObject } from "@/runtime/extendModVariableContext";
 import { unary } from "lodash";
 import { toExpression } from "@/utils/expressionUtils";
 import { showModal } from "@/contentScript/modalDom";
+import { isLoadedInIframe } from "@/utils/iframeUtils";
 
 jest.mock("@/contentScript/modalDom");
 jest.mock("@/contentScript/sidebarController");
 jest.mock("@/platform/panels/panelController");
 
+jest.mock("@/utils/iframeUtils");
+
 const displayTemporaryInfoBlock = new DisplayTemporaryInfo();
 const renderer = new DocumentRenderer();
 
 describe("DisplayTemporaryInfo", () => {
   beforeEach(() => {
+    jest.mocked(isLoadedInIframe).mockReturnValue(false);
+
     blockRegistry.clear();
     blockRegistry.register([
       echoBrick,
@@ -197,6 +202,34 @@ describe("DisplayTemporaryInfo", () => {
     });
   });
 
+  test("it errors from frame", async () => {
+    jest.mocked(isLoadedInIframe).mockReturnValue(true);
+
+    const config = getExampleBrickConfig(renderer.id);
+    const pipeline = {
+      id: displayTemporaryInfoBlock.id,
+      config: {
+        title: "Test Temp Panel",
+        body: toExpression("pipeline", [{ id: renderer.id, config }]),
+        location: "panel",
+        isRootAware: true,
+      },
+    };
+
+    const extensionId = uuidv4();
+
+    const options = {
+      ...testOptions("v3"),
+      logger: new ConsoleLogger({
+        extensionId,
+      }),
+    };
+
+    await expect(
+      reducePipeline(pipeline, simpleInput({}), options),
+    ).rejects.toThrow("Cannot show sidebar in a frame");
+  });
+
   test("requires target for popover", async () => {
     const config = getExampleBrickConfig(renderer.id);
     const pipeline = {

src/contentScript/ephemeralPanel.ts

@@ -43,6 +43,7 @@ import type { Location } from "@/types/starterBrickTypes";
 import { getThisFrame } from "webext-messenger";
 import { expectContext } from "@/utils/expectContext";
 import { showModal } from "@/contentScript/modalDom";
+import { isLoadedInIframe } from "@/utils/iframeUtils";
 
 export async function createFrameSource(
   nonce: string,
@@ -82,6 +83,13 @@ export async function ephemeralPanel({
 }: TemporaryPanelDefinition): Promise<JsonObject> {
   expectContext("contentScript");
 
+  if (location === "panel" && isLoadedInIframe()) {
+    // Validate before registerEmptyTemporaryPanel to avoid an uncaught promise rejection
+    throw new BusinessError(
+      "Cannot show sidebar in a frame. To use the sidebar, set the target to Top-level Frame",
+    );
+  }
+
   const nonce = uuidv4();
   let onReady: (() => void) | undefined;