From 5d4f81a007d83299a14a76788aa6813f7f190085 Mon Sep 17 00:00:00 2001 From: Colin McDonnell Date: Fri, 23 Jan 2026 06:33:29 +0000 Subject: [PATCH] Improve logging on resovelBody --- entry | 22 +++++++++++++++++++--- utils/body.ts | 24 +++++++++++++++++++++--- 2 files changed, 40 insertions(+), 6 deletions(-) diff --git a/entry b/entry index df4bda3..aa3ad8d 100755 --- a/entry +++ b/entry @@ -157786,10 +157786,26 @@ function hasImages(body) { } async function resolveBody(ctx) { const body = ctx.event.body; - if (!hasImages(body)) return body ?? null; + log.info(`[resolveBody] trigger: ${ctx.event.trigger}`); + log.info(`[resolveBody] original body (first 200 chars): ${body?.substring(0, 200)}`); + log.info(`[resolveBody] hasImages: ${hasImages(body)}`); + if (!hasImages(body)) { + log.info("[resolveBody] no images detected, returning original body"); + return body ?? null; + } + log.info("[resolveBody] images detected, fetching body_html from GitHub API..."); const bodyHtml = await fetchBodyHtml(ctx); - if (!bodyHtml) return body ?? null; - return turndown.turndown(bodyHtml); + log.info(`[resolveBody] bodyHtml received: ${bodyHtml ? "yes" : "no"}`); + if (bodyHtml) { + log.info(`[resolveBody] bodyHtml (first 500 chars): ${bodyHtml.substring(0, 500)}`); + } + if (!bodyHtml) { + log.info("[resolveBody] no bodyHtml returned, returning original body"); + return body ?? null; + } + const resolved = turndown.turndown(bodyHtml); + log.info(`[resolveBody] resolved markdown (first 500 chars): ${resolved.substring(0, 500)}`); + return resolved; } async function fetchBodyHtml(ctx) { const event = ctx.event; diff --git a/utils/body.ts b/utils/body.ts index 1442bc0..1dc4a79 100644 --- a/utils/body.ts +++ b/utils/body.ts @@ -2,6 +2,7 @@ import TurndownService from "turndown"; import type { PayloadEvent } from "../external.ts"; import type { OctokitWithPlugins } from "./github.ts"; import type { RepoData } from "./repoData.ts"; +import { log } from "./cli.ts"; const turndown = new TurndownService(); @@ -25,13 +26,30 @@ interface ResolveBodyContext { export async function resolveBody(ctx: ResolveBodyContext): Promise { const body = ctx.event.body; + log.info(`[resolveBody] trigger: ${ctx.event.trigger}`); + log.info(`[resolveBody] original body (first 200 chars): ${body?.substring(0, 200)}`); + log.info(`[resolveBody] hasImages: ${hasImages(body)}`); + // pass through if no images - no API call needed - if (!hasImages(body)) return body ?? null; + if (!hasImages(body)) { + log.info("[resolveBody] no images detected, returning original body"); + return body ?? null; + } + log.info("[resolveBody] images detected, fetching body_html from GitHub API..."); const bodyHtml = await fetchBodyHtml(ctx); - if (!bodyHtml) return body ?? null; + log.info(`[resolveBody] bodyHtml received: ${bodyHtml ? "yes" : "no"}`); + if (bodyHtml) { + log.info(`[resolveBody] bodyHtml (first 500 chars): ${bodyHtml.substring(0, 500)}`); + } + if (!bodyHtml) { + log.info("[resolveBody] no bodyHtml returned, returning original body"); + return body ?? null; + } - return turndown.turndown(bodyHtml); + const resolved = turndown.turndown(bodyHtml); + log.info(`[resolveBody] resolved markdown (first 500 chars): ${resolved.substring(0, 500)}`); + return resolved; } async function fetchBodyHtml(ctx: ResolveBodyContext): Promise {