show model name in footer, drop pullfrog.com link (#484)

* show model name in footer, drop pullfrog.com link

add model slug to buildPullfrogFooter so every Pullfrog comment
displays the active model (e.g. "Using `Big Pickle` (free)" or
"Using `Claude Opus`"). remove the pullfrog.com link from all footers.

Made-with: Cursor

* reject <br/> tags in comment bodies, add prompt guidance

add runtime validation in addFooter that throws if <br/> is followed
by a non-blank line (breaks GitHub heading rendering). the agent sees
the error and retries with clean markdown. also update Summarize mode
prompt to explicitly forbid <br/> tags.

Made-with: Cursor

* fix <br/> guidance: move to event instructions, clarify blank line rule

the formatting rule belongs in DEFAULT_PR_SUMMARY_INSTRUCTIONS (event
instructions), not the Summarize mode prompt. clarify that <br/> must
always be followed by a blank line before headings.

Made-with: Cursor

* generalize block-level HTML rule in summary instructions

add a prominent top-level rule about requiring blank lines between ALL
block-level HTML elements and markdown syntax, not just <br/>.

Made-with: Cursor

* move model to toolState instead of threading through params

model is set once at startup and read everywhere — it belongs on
toolState, not threaded as a separate param through 8 call sites.
postCleanup runs without toolState so it just omits the model label.

Made-with: Cursor

* update models.dev snapshot (openai latest changed)

Made-with: Cursor

* add comment to models snapshot test explaining its purpose

Made-with: Cursor

mcp/comment.ts

@@ -55,6 +55,7 @@ export const LEAPING_INTO_ACTION_PREFIX = "Leaping into action";
 interface BuildCommentFooterParams {
   octokit?: OctokitWithPlugins | undefined;
   customParts?: string[] | undefined;
+  model?: string | undefined;
 }
 
 async function buildCommentFooter(params: BuildCommentFooterParams): Promise<string> {
@@ -77,17 +78,14 @@ async function buildCommentFooter(params: BuildCommentFooterParams): Promise<str
     }
   }
 
-  const footerParams = {
+  return buildPullfrogFooter({
     triggeredBy: true,
     workflowRun: runId
       ? { owner: repoContext.owner, repo: repoContext.name, runId, jobId }
       : undefined,
-  };
-
-  if (params.customParts && params.customParts.length > 0) {
-    return buildPullfrogFooter({ ...footerParams, customParts: params.customParts });
-  }
-  return buildPullfrogFooter(footerParams);
+    customParts: params.customParts,
+    model: params.model,
+  });
 }
 
 function buildImplementPlanLink(
@@ -102,11 +100,17 @@ function buildImplementPlanLink(
 
 export interface AddFooterCtx {
   octokit?: OctokitWithPlugins | undefined;
+  toolState?: { model?: string | undefined } | undefined;
 }
 
 export async function addFooter(ctx: AddFooterCtx, body: string): Promise<string> {
+  if (/<br\s*\/?>[ \t]*\n(?!\s*\n)/i.test(body)) {
+    throw new Error(
+      "body contains <br/> followed by a non-blank line, which breaks GitHub markdown rendering. always add a blank line after <br/> tags."
+    );
+  }
   const bodyWithoutFooter = stripExistingFooter(fixDoubleEscapedString(body));
-  const footer = await buildCommentFooter({ octokit: ctx.octokit });
+  const footer = await buildCommentFooter({ octokit: ctx.octokit, model: ctx.toolState?.model });
   return `${bodyWithoutFooter}${footer}`;
 }
 
@@ -262,6 +266,7 @@ export async function reportProgress(
     const footer = await buildCommentFooter({
       octokit: ctx.octokit,
       customParts,
+      model: ctx.toolState.model,
     });
     const bodyWithFooter = `${bodyWithoutFooter}${footer}`;
 
@@ -299,6 +304,7 @@ export async function reportProgress(
     const footer = await buildCommentFooter({
       octokit: ctx.octokit,
       customParts,
+      model: ctx.toolState.model,
     });
     const bodyWithFooter = `${bodyWithoutFooter}${footer}`;
 
@@ -359,6 +365,7 @@ export async function reportProgress(
     const footer = await buildCommentFooter({
       octokit: ctx.octokit,
       customParts,
+      model: ctx.toolState.model,
     });
     const bodyWithPlanLink = `${bodyWithoutFooter}${footer}`;
 

mcp/pr.ts

@@ -21,6 +21,7 @@ function buildPrBodyWithFooter(ctx: ToolContext, body: string): string {
     workflowRun: ctx.runId
       ? { owner: ctx.repo.owner, repo: ctx.repo.name, runId: ctx.runId, jobId: ctx.jobId }
       : undefined,
+    model: ctx.toolState.model,
   });
 
   const bodyWithoutFooter = stripExistingFooter(fixDoubleEscapedString(body));

mcp/review.ts

@@ -264,6 +264,7 @@ async function createAndSubmitWithFooter(
       ? { owner: ctx.repo.owner, repo: ctx.repo.name, runId: ctx.runId, jobId: ctx.jobId }
       : undefined,
     customParts,
+    model: ctx.toolState.model,
   });
 
   return ctx.octokit.rest.pulls.submitReview({

mcp/server.ts

@@ -89,6 +89,7 @@ export interface ToolState {
   existingSummaryCommentId?: number;
   output?: string;
   usageEntries: AgentUsage[];
+  model?: string | undefined;
 }
 
 interface InitToolStateParams {