diff --git a/lib/ads/formats.ts b/lib/ads/formats.ts
index a0114683..391c896a 100644
--- a/lib/ads/formats.ts
+++ b/lib/ads/formats.ts
@@ -15,6 +15,12 @@ export const AD_FORMATS = [
export type AdFormatId = (typeof AD_FORMATS)[number]["id"];
export const AD_FORMAT_IDS = AD_FORMATS.map((f) => f.id) as AdFormatId[];
+// Sizes offered to publishers on the Monetize page — the ones they can copy an
+// embed for and install. A subset of AD_FORMATS that grows as each size is
+// surfaced (one PR per size). Keep the medium rectangle first: it's the safe
+// default the auto-installer falls back to.
+export const PUBLISHER_FORMAT_IDS: AdFormatId[] = ["banner_300x250", "banner_728x90"];
+
export function formatSpec(id: AdFormatId) {
return AD_FORMATS.find((f) => f.id === id) ?? AD_FORMATS[0];
}
diff --git a/lib/github/install-ad.ts b/lib/github/install-ad.ts
index 426165e1..9d084f62 100644
--- a/lib/github/install-ad.ts
+++ b/lib/github/install-ad.ts
@@ -134,6 +134,36 @@ function injectBeforeBodyClose(content: string, embed: string, path: string): st
return updated;
}
+// Inject right after the opening tag — the "right place" for a
+// leaderboard, which reads best across the top of the page rather than jammed
+// at the very bottom before . Best-effort: callers fall back to
+// injectBeforeBodyClose when there's no tag (e.g. a React fragment).
+function injectAfterBodyOpen(content: string, embed: string, path: string): string | null {
+ const match = content.match(/]*>/i);
+ if (!match || match.index == null) return null;
+ const openEnd = match.index + match[0].length;
+ const lineStart = content.lastIndexOf("\n", match.index) + 1;
+ const indent = content.slice(lineStart, match.index).match(/^\s*/)?.[0] ?? "";
+ let updated = `${content.slice(0, openEnd)}\n${indent} ${embed}${content.slice(openEnd)}`;
+ if (isJsx(path) && /.
+const TOP_PLACED_FORMATS = new Set(["banner_728x90"]);
+
+// Choose where a format's embed lands. Leaderboards go up top; everything else
+// (rectangle, mobile, text link) drops in before . Always falls back to
+// the other strategy so a missing / never blocks the install.
+function injectEmbed(content: string, embed: string, path: string, format: string): string | null {
+ if (TOP_PLACED_FORMATS.has(format)) {
+ return injectAfterBodyOpen(content, embed, path) ?? injectBeforeBodyClose(content, embed, path);
+ }
+ return injectBeforeBodyClose(content, embed, path);
+}
+
export async function installAdEmbed(input: InstallAdInput): Promise {
const format = input.format ?? DEFAULT_FORMAT;
const repoMeta = await getRepo({ token: input.token, owner: input.owner, repo: input.repo });
@@ -171,9 +201,9 @@ export async function installAdEmbed(input: InstallAdInput): Promise tag in ${file.path}.` };
+ return { status: "noop", path: file.path, detail: `No tag in ${file.path}.` };
}
// Patch the site's CSP so the browser can load /ad.js, reach /api/ads/serve,
diff --git a/tests/contract/install-ad.test.ts b/tests/contract/install-ad.test.ts
index 590a8d1a..63a57d2f 100644
--- a/tests/contract/install-ad.test.ts
+++ b/tests/contract/install-ad.test.ts
@@ -130,6 +130,29 @@ describe("installAdEmbed", () => {
expect(written).toContain("next.config.ts");
});
+ it("places a leaderboard at the top of the page (after ), not before ", async () => {
+ github.files.set(
+ "app/layout.tsx",
+ "export default function RootLayout({ children }) {\n return {children};\n}\n",
+ );
+
+ await installAdEmbed({
+ token: "token",
+ owner: "owner",
+ repo: "repo",
+ slotId: "slot-abc",
+ format: "banner_728x90",
+ });
+
+ const write = github.putFile.mock.calls.find((c) => c[0].path === "app/layout.tsx");
+ expect(write).toBeDefined();
+ const content = write![0].contentUtf8 as string;
+ // The embed carries the requested format and lands before , i.e. right
+ // after rather than at the very bottom of the page.
+ expect(content).toContain('data-format="banner_728x90"');
+ expect(content.indexOf("data-cp-ad")).toBeLessThan(content.indexOf(""));
+ });
+
it("no-ops when the embed exists and no CSP needs changes", async () => {
github.files.set(
"app/layout.tsx",