@cyanheads/hn-mcp-server 0.6.0 → 0.7.0

package/dist/mcp-server/tools/definitions/search-content.tool.js

@@ -5,9 +5,11 @@
 import { tool, z } from '@cyanheads/mcp-ts-core';
 import { JsonRpcErrorCode } from '@cyanheads/mcp-ts-core/errors';
 import { escapeHighlight, escapeInline, escapeQuoted, quoteBody, } from '../markdown-escape.js';
-import { byteLength, cutPointer, cutToBytes, fitsBudget, largestFittingLimit, measureResponse, RESPONSE_BUDGET_TEXT, } from '../response-budget.js';
 import { servedAtCap } from '../served-at-cap.js';
+import { usernameInput } from '../username-input.js';
 import { dateBoundToEpochMs, dateRangeToEpochSeconds, decodeHtmlEntities, extractDomain, getHnService, normalizeUrl, stripHtml, stripHtmlPreservingEm, } from '../../../services/hn/hn-service.js';
+/** `count` when the caller omits it. */
+const COUNT_DEFAULT = 10;
 /** Most hits one call serves; a larger `count` is served at this cap. */
 const COUNT_CAP = 50;
 /** Largest `count` accepted; anything above it is rejected. */
@@ -104,9 +106,13 @@ export function stripUrlPrefix(query) {
 const searchCriteria = z
     .object({
     tags: z.enum(SEARCH_TAGS).optional().describe('Content-type filter.'),
-    author: z.string().optional().describe('Author filter — the username matched, trimmed.'),
+    author: z
+        .string()
+        .optional()
+        .describe('Author filter — the username as searched: trimmed, with one leading @ dropped.'),
     storyId: z.number().optional().describe('Discussion filter: a story or poll root id.'),
     minPoints: z.number().optional().describe('Minimum score.'),
+    minComments: z.number().optional().describe('Minimum comment count.'),
     dateRange: z
         .object({
         start: z
@@ -126,7 +132,7 @@ const searchCriteria = z
         .describe('Hits per page sent: count as passed, or 50 when count was above that cap.'),
     view: z.enum(['full', 'compact']).describe('Projection applied to each hit.'),
 })
-    .describe('The criteria this search ran with: each filter that was set, as sent — author trimmed, dateRange bounds as the exclusive UTC instants applied — then sort, count, and view, defaults included. Unset filters are absent. The searched query is in query.');
+    .describe('The criteria this search ran with: each filter that was set, as sent — author as searched (trimmed, one leading @ dropped), dateRange bounds as the exclusive UTC instants applied — then sort, count, and view, defaults included. Unset filters are absent. The searched query is in query.');
 /** A whole Unix second as ISO 8601 UTC, e.g. `2024-05-05T08:00:00Z`. */
 const isoSeconds = (seconds) => new Date(seconds * 1000).toISOString().replace('.000Z', 'Z');
 /** One trailer line for the criteria; `author`, the one free-text value, is escaped inside its quotes. */
@@ -140,6 +146,7 @@ function renderCriteria(c) {
         c.author != null ? `author: "${escapeQuoted(c.author)}"` : null,
         c.storyId != null ? `storyId: ${c.storyId}` : null,
         c.minPoints != null ? `minPoints: ${c.minPoints}` : null,
+        c.minComments != null ? `minComments: ${c.minComments}` : null,
         bounds ? `dateRange: ${bounds.filter(Boolean).join(', ')}` : null,
         `sort: ${c.sort}`,
         `count: ${c.count}`,
@@ -158,6 +165,8 @@ function appliedFilters(input) {
         filters.push('storyId');
     if (input.minPoints != null)
         filters.push('minPoints');
+    if (input.minComments != null)
+        filters.push('minComments');
     if (input.dateRange?.start || input.dateRange?.end)
         filters.push('dateRange');
     return filters;
@@ -168,16 +177,23 @@ function appliedFilters(input) {
  * that ceiling. An empty page past the end names the last valid page when
  * Algolia reports one; past the ceiling Algolia answers `nbPages: 0`, so the
  * last page is unknown and the notice resets to page 0. Only a first page with
- * no hits is a zero-match search, which names what to relax. After a count
- * above the cap, a page lever also names the count it counts in.
+ * no hits is a zero-match search, which names what to relax.
+ *
+ * `page` counts in pages of `count`, so every page lever names the count the
+ * search ran at unless that is the default — a caller who sends the lever as
+ * written, without count, would otherwise page at the default and repeat or
+ * skip hits. Raising count is offered only on page 0, where the larger page
+ * still starts at hit 0; on a later page it would move every page boundary.
  */
 function pageStateSentence(facts) {
     const { shown, page, totalHits, totalPages } = facts;
-    const pageLever = (n) => `${facts.overCap ? `count: ${facts.count}, ` : ''}page: ${n}`;
+    const pageLever = (n) => `${facts.count === COUNT_DEFAULT ? '' : `count: ${facts.count}, `}page: ${n}`;
     if (shown > 0) {
         const total = `${facts.totalHitsExact === false ? 'about ' : ''}${totalHits}`;
         if (page + 1 < totalPages) {
-            const raiseCount = facts.count < COUNT_CAP ? `, or raise count (max ${COUNT_CAP})` : '';
+            const raiseCount = page === 0 && facts.count < COUNT_CAP
+                ? `, or keep page: 0 and raise count (max ${COUNT_CAP})`
+                : '';
             return `Showing ${shown} of ${total} hits (page ${page} of ${totalPages}). Pass ${pageLever(page + 1)} for more${raiseCount}.`;
         }
         return totalHits > HIT_CEILING
@@ -186,12 +202,12 @@ function pageStateSentence(facts) {
     }
     if (page > 0) {
         if (totalPages === 0) {
-            return `Page ${page} is past the last page Algolia serves for this search (at most 1,000 hits per search), where totalHits and totalPages read 0. Pass page: 0 to start over; if that also returns nothing, the search itself matches nothing.`;
+            return `Page ${page} is past the last page Algolia serves for this search (at most 1,000 hits per search), where totalHits and totalPages read 0. Pass ${pageLever(0)} to start over; if that also returns nothing, the search itself matches nothing.`;
         }
         /** With one page, the last page is page 0, so one lever covers both. */
         const lever = totalPages === 1
             ? `Pass ${pageLever(0)} for that page.`
-            : `Pass ${pageLever(totalPages - 1)} for the last page, or page: 0 to start over.`;
+            : `Pass ${pageLever(totalPages - 1)} for the last page, or ${pageLever(0)} to start over.`;
         return `Page ${page} is past the last page of results (${totalHits} hits across ${totalPages} page${totalPages === 1 ? '' : 's'}). ${lever}`;
     }
     const filters = appliedFilters(facts);
@@ -237,29 +253,15 @@ function causeSentences(facts) {
     }
     return sentences;
 }
-/** The sentence disclosing how the page was fitted to the response budget. */
-function budgetSentence({ highlightsTextOmitted, cutIds }) {
-    const budget = `To stay within the ${RESPONSE_BUDGET_TEXT}-byte response budget,`;
-    const omitted = 'highlights.text is omitted from every hit';
-    if (cutIds.length === 0)
-        return `${budget} ${omitted}; matchedWords still lists the matched terms.`;
-    const cut = `the text of ${cutIds.length === 1 ? 'hit' : 'hits'} ${cutIds.join(', ')} is cut short`;
-    return `${budget} ${highlightsTextOmitted ? `${omitted} and ` : ''}${cut}; hn_get_thread with a cut hit's id returns its full text, and view: "compact" or a smaller count avoids the cut.`;
-}
 /**
  * Compose the one `notice` string every search writes. `notice` is last-wins,
  * so each source goes through here: the page-state sentence first, then the
- * cause sentences in input order — query, tags, author, storyId, count — then
- * the budget sentence for a page fitted to the response budget. None replaces
- * another. Undefined when nothing applies — the last page of an uncapped,
- * non-empty search that fit.
+ * cause sentences in input order — query, tags, author, storyId, count. None
+ * replaces another. Undefined when nothing applies — the last page of an
+ * uncapped, non-empty search.
  */
 export function composeSearchNotice(facts) {
-    const sentences = [
-        pageStateSentence(facts),
-        ...causeSentences(facts),
-        facts.budget && budgetSentence(facts.budget),
-    ].filter((sentence) => sentence != null);
+    const sentences = [pageStateSentence(facts), ...causeSentences(facts)].filter((sentence) => sentence != null);
     return sentences.length > 0 ? sentences.join(' ') : undefined;
 }
 /**
@@ -279,26 +281,27 @@ function decodeHighlightTitle(title) {
  * (comment_text, else story_text — the order `text` reads them in; a job's
  * job_text is never highlighted) — and the deduplicated
  * union of matched words across all surfaced fields. Returns undefined when
- * nothing matched.
+ * Algolia highlighted no field and reported no matched word. That is Algolia's
+ * answer, not a finding about the hit: it marks every field `matchLevel: none`
+ * on some hits whose body holds every query term.
  *
  * Titles are not HTML, though some arrive entity-encoded (`&lt;details&gt;`,
  * `&#34;`), so the title snippet is entity-decoded between its `<em>` markers
  * and never tag-stripped. The body highlight, the whole matched body, is
  * stripped of its HTML with the markers kept.
  *
- * `includeBody` is false under the compact projection, and on a full page over
- * the response budget, both of which drop the body highlight — the field that
- * repeats a hit's full body text.
+ * `includeBody` is false under the compact projection, which drops the body
+ * highlight — the field that repeats a hit's full body text.
  */
 function extractHighlights(hit, includeBody) {
     const h = hit._highlightResult;
     if (!h)
         return;
-    const matched = (v) => v != null && v.matchLevel !== 'none';
-    const title = matched(h.title) ? h.title.value : undefined;
-    const textHl = matched(h.comment_text)
+    const highlighted = (v) => v != null && v.matchLevel !== 'none';
+    const title = highlighted(h.title) ? h.title.value : undefined;
+    const textHl = highlighted(h.comment_text)
         ? h.comment_text
-        : matched(h.story_text)
+        : highlighted(h.story_text)
             ? h.story_text
             : undefined;
     const text = includeBody && textHl ? stripHtmlPreservingEm(textHl.value) : undefined;
@@ -334,14 +337,18 @@ const searchHit = z
         .number()
         .optional()
         .describe("Parent story ID for comment hits; equals `id` for story hits. Absent for polls, poll options, and jobs — a poll's own `id` is its discussion root."),
-    text: z
+    parentId: z
+        .number()
+        .optional()
+        .describe("The item a comment hit replies to: its story's id for a top-level comment, otherwise the parent comment's id — hn_get_thread with this id reads it. Present on comment hits only."),
+    storyUrl: z
         .string()
         .optional()
-        .describe(`Body text, HTML stripped: a comment's text, a text post's body, or a job's description. A job description is returned but not keyword-searched or highlighted. Cut short when the page would otherwise exceed the ${RESPONSE_BUDGET_TEXT}-byte response budget, with textTruncated set. Absent when the hit has no body, and always absent under view "compact" — call hn_get_thread with this id to read it.`),
-    textTruncated: z
-        .boolean()
+        .describe("External link URL of a comment hit's story. Absent when that story is a text post with no link, and on every hit that is not a comment."),
+    text: z
+        .string()
         .optional()
-        .describe(`True when text is a prefix of the body, cut short to keep the page within the ${RESPONSE_BUDGET_TEXT}-byte response budget — hn_get_thread with this id returns it whole. Absent when text is whole.`),
+        .describe(`Body text, HTML stripped: a comment's text, a text post's body, or a job's description. A job description is returned but not keyword-searched or highlighted. Absent when the hit has no body, and always absent under view "compact" — call hn_get_thread with this id to read it.`),
     highlights: z
         .object({
         title: z
@@ -351,25 +358,26 @@ const searchHit = z
         text: z
             .string()
             .optional()
-            .describe('The matched body in full, not a snippet (comment_text or story_text), HTML stripped, with matched terms wrapped in `<em>…</em>` — the same text as `text` plus its markers. A literal `<em>` typed into the body is indistinguishable from a marker. Absent when the body did not match, when highlightsTextOmitted is true, and always under view "compact" — matchedWords still lists what matched.'),
+            .describe('The matched body in full, not a snippet (comment_text or story_text), HTML stripped, with matched terms wrapped in `<em>…</em>` — the same text as `text` plus its markers. A literal `<em>` typed into the body is indistinguishable from a marker. Absent when Algolia returned no highlight for the body, which does not mean the body lacks the query terms: Algolia leaves some bodies that hold every term unhighlighted. Always absent under view "compact", where matchedWords still lists the matched terms.'),
         matchedWords: z
             .array(z.string())
             .describe('Deduplicated union of matched terms across all searchable fields (title, url, author, comment_text, story_text, story_title).'),
     })
         .optional()
-        .describe('Algolia per-field highlight metadata showing which terms matched and where. Absent when no fields produced a match.'),
+        .describe('Algolia per-field highlight metadata showing which terms matched and where. Absent when Algolia returned no highlight and no matched terms for any field, which does not mean the hit lacks the query terms: Algolia leaves some hits that hold every term unhighlighted.'),
 })
     .describe('A single Algolia search hit (story, comment, poll, poll option, or job).');
 /**
- * Project an Algolia hit into a search hit. `text` carries its body; under
- * `bodyHighlight` its highlights carry the body too — the matched body in full,
- * with markers.
+ * Project an Algolia hit into a search hit. Under `includeBody`, `text`
+ * carries its body and its highlights carry the body too — the matched body in
+ * full, with markers.
  */
-function projectHit(hit, text, bodyHighlight) {
-    const rawText = text ? (hit.comment_text ?? hit.story_text ?? hit.job_text) : undefined;
+function projectHit(hit, includeBody) {
+    const rawText = includeBody ? (hit.comment_text ?? hit.story_text ?? hit.job_text) : undefined;
     const url = normalizeUrl(hit.url);
     const domain = extractDomain(url);
-    const highlights = extractHighlights(hit, bodyHighlight);
+    const storyUrl = normalizeUrl(hit.story_url);
+    const highlights = extractHighlights(hit, includeBody);
     const type = hitType(hit._tags);
     return {
         id: Number(hit.objectID),
@@ -383,6 +391,8 @@ function projectHit(hit, text, bodyHighlight) {
         createdAt: hit.created_at,
         storyTitle: hit.story_title != null ? decodeHtmlEntities(hit.story_title) : undefined,
         storyId: hit.story_id ?? undefined,
+        ...(hit.parent_id != null && { parentId: hit.parent_id }),
+        ...(storyUrl && { storyUrl }),
         text: rawText ? stripHtml(rawText) || undefined : undefined,
         ...(highlights && { highlights }),
     };
@@ -394,9 +404,6 @@ const searchOutput = z.object({
         .optional()
         .describe('The query that was searched — for a URL query, without its leading scheme and www. Absent for a filter-only search.'),
 });
-const searchTrailer = {
-    criteria: { render: renderCriteria },
-};
 /**
  * Render highlight metadata as a `> match — …` footer, surfacing each highlights
  * field separately so structured consumers and the LLM both see what matched.
@@ -414,10 +421,31 @@ function renderHighlights(hl) {
         parts.push(`body: ${hl.text}`);
     return parts.length ? `\n\n${quoteBody(`match — ${parts.join(' | ')}`)}` : '';
 }
-/** A hit's quoted body, and after a cut body the unquoted pointer to the whole text, a blank line apart. */
+/** A hit's body, quoted on the lines under its meta line and links. */
 function renderBody(h) {
-    const text = h.text ? `\n${quoteBody(h.text)}` : '';
-    return h.textTruncated ? `${text}\n\n${cutPointer(h.id)}` : text;
+    return h.text ? `\n${quoteBody(h.text)}` : '';
+}
+/**
+ * A hit's meta line: id, type, and the item a comment replies to, in the order
+ * `hn_get_thread` opens its meta line with, then author, score, comment count,
+ * and time. Each field renders when present, whatever the hit's type.
+ */
+function metaLine(h) {
+    return [
+        `id:${h.id}`,
+        h.type ? `type:${h.type}` : null,
+        h.parentId != null ? `parent:${h.parentId}` : null,
+        h.author,
+        h.points != null ? `${h.points} pts` : null,
+        h.numComments != null ? `${h.numComments} comments` : null,
+        h.createdAt,
+    ]
+        .filter(Boolean)
+        .join(' | ');
+}
+/** The hit's own link, then its story's, each on a line of its own under the meta line. */
+function renderLinks(h) {
+    return [h.url, h.storyUrl].map((link) => (link ? `\n${link}` : '')).join('');
 }
 function formatSearch(result) {
     if (result.hits.length === 0) {
@@ -429,51 +457,23 @@ function formatSearch(result) {
         ];
     }
     const lines = result.hits.map((h) => {
-        const typeToken = h.type ? `type:${h.type}` : null;
+        const meta = metaLine(h);
+        const links = renderLinks(h);
         const hlLine = h.highlights ? renderHighlights(h.highlights) : '';
         if (h.type === 'pollopt') {
             // A poll option's search record carries no text and no poll; hn_get_thread has both.
-            const meta = [
-                `id:${h.id}`,
-                typeToken,
-                h.author,
-                h.points != null ? `${h.points} pts` : null,
-                h.createdAt,
-            ]
-                .filter(Boolean)
-                .join(' | ');
-            return `### Poll option by ${h.author}\n${meta}\nOption text and its poll: hn_get_thread with itemId ${h.id}${hlLine}`;
+            return `### Poll option by ${h.author}\n${meta}${links}\nOption text and its poll: hn_get_thread with itemId ${h.id}${hlLine}`;
         }
         if (h.title) {
             // Story result — Algolia returns storyId === id for stories, so suppress the parent ref unless it actually differs or a parent title is set.
             const parentRef = (h.storyId != null && h.storyId !== h.id) || h.storyTitle
                 ? ` | story:"${escapeQuoted(h.storyTitle ?? '?')}"#${h.storyId ?? '?'}`
                 : '';
-            const meta = [
-                `id:${h.id}`,
-                typeToken,
-                h.author,
-                h.points != null ? `${h.points} pts` : null,
-                h.numComments != null ? `${h.numComments} comments` : null,
-                h.createdAt,
-            ]
-                .filter(Boolean)
-                .join(' | ');
             const domain = h.domain ? ` (${h.domain})` : '';
-            const url = h.url ? `\n${h.url}` : '';
-            return `### ${escapeInline(h.title)}${domain}\n${meta}${parentRef}${url}${renderBody(h)}${hlLine}`;
+            return `### ${escapeInline(h.title)}${domain}\n${meta}${parentRef}${links}${renderBody(h)}${hlLine}`;
         }
         // Comment result — parent context in heading.
-        const meta = [
-            `id:${h.id}`,
-            typeToken,
-            h.author,
-            h.points != null ? `${h.points} pts` : null,
-            h.createdAt,
-        ]
-            .filter(Boolean)
-            .join(' | ');
-        return `### Comment on "${escapeQuoted(h.storyTitle ?? 'unknown')}" (story id:${h.storyId ?? '?'})\n${meta}${renderBody(h)}${hlLine}`;
+        return `### Comment on "${escapeQuoted(h.storyTitle ?? 'unknown')}" (story id:${h.storyId ?? '?'})\n${meta}${links}${renderBody(h)}${hlLine}`;
     });
     const header = result.query != null
         ? `## "${escapeQuoted(result.query)}" — search results`
@@ -481,14 +481,14 @@ function formatSearch(result) {
     return [{ type: 'text', text: `${header}\n\n${lines.join('\n\n')}` }];
 }
 export const searchHn = tool('hn_search_content', {
-    description: 'Search Hacker News stories, comments, polls, and jobs via Algolia — by keyword, by filters alone, or both. Filterable by content type, author, parent story, date range, and minimum points.',
+    description: 'Search Hacker News stories, comments, polls, and jobs via Algolia — by keyword, by filters alone, or both. Filterable by content type, author, parent story, date range, minimum points, and minimum comment count.',
     annotations: { readOnlyHint: true },
     errors: [
         {
             reason: 'missing_query_or_filter',
             code: JsonRpcErrorCode.InvalidParams,
             when: 'Neither a query nor any filter was supplied, so there is nothing to search by.',
-            recovery: 'Pass a query, or at least one filter: tags, author, storyId, minPoints, or a dateRange bound. sort, count, page, and view do not count as filters.',
+            recovery: 'Pass a query, or at least one filter: tags, author, storyId, minPoints, minComments, or a dateRange bound. sort, count, page, and view do not count as filters.',
             severity: 'notice',
         },
         {
@@ -505,6 +505,13 @@ export const searchHn = tool('hn_search_content', {
             recovery: 'Drop minPoints to search comments or jobs, or search a scored type (story, poll, ask_hn, show_hn, front_page) to filter by points.',
             severity: 'notice',
         },
+        {
+            reason: 'min_comments_unscored_type',
+            code: JsonRpcErrorCode.InvalidParams,
+            when: 'minComments was combined with tags "comment" or "job", record types Algolia stores without a comment count.',
+            recovery: 'Drop minComments to search comments or jobs, or search a type with a comment count (story, poll, ask_hn, show_hn, front_page) to filter by comments.',
+            severity: 'notice',
+        },
         {
             reason: 'count_page_unaligned',
             code: JsonRpcErrorCode.InvalidParams,
@@ -558,17 +565,14 @@ export const searchHn = tool('hn_search_content', {
             .trim()
             .min(1, 'blank after trimming whitespace — pass one or more search terms, or omit query for a filter-only search.')
             .optional()
-            .describe('Search terms. Supports simple keywords — Algolia handles stemming and relevance, and a hit must contain every term. Matching is typo-tolerant and the last term also matches as a prefix (zig matches Zigbee); wrap a term in double quotes to match it exactly ("zig"), or put - before one to exclude it ("zig" -devlog). Trimmed before searching; blank or whitespace-only input is rejected. To find discussions of a page, pass its URL alone: a query with no spaces that starts with http://, https://, or www. is searched without that prefix, so submissions under either scheme, with or without www., all match. Omit for a filter-only search, which needs at least one of tags, author, storyId, minPoints, or a dateRange bound.'),
+            .describe('Search terms. Supports simple keywords — Algolia handles stemming and relevance, and a hit must contain every term. Matching is typo-tolerant and the last term also matches as a prefix (zig matches Zigbee); wrap a term in double quotes to match it exactly ("zig"), or put - before one to exclude it ("zig" -devlog). Trimmed before searching; blank or whitespace-only input is rejected. To find discussions of a page, pass its URL alone: a query with no spaces that starts with http://, https://, or www. is searched without that prefix, so submissions under either scheme, with or without www., all match. Omit for a filter-only search, which needs at least one of tags, author, storyId, minPoints, minComments, or a dateRange bound.'),
         tags: z
             .enum(SEARCH_TAGS)
             .optional()
             .describe('Filter results by content type: "story", "comment", "poll", or "job", or the story subsets "ask_hn", "show_hn", and "front_page". "front_page" is the HN front page right now (about 30 stories); no filter selects stories from past front pages. The nearest proxy is "story" with dateRange and minPoints — without a query, the default relevance sort ranks by points. Omit to search all types, which also returns poll options (type "pollopt"): records with only an id, author, vote count, and date — hn_get_thread reads an option\'s text and poll.'),
-        author: z
-            .string()
-            .trim()
-            .min(1, 'blank after trimming whitespace — omit author to search all authors.')
+        author: usernameInput('omit author to search all authors.')
             .optional()
-            .describe(`Filter results to a specific author — useful for finding a user's posts on a topic. Matches the HN username exactly and is case-sensitive, so a re-cased name finds nothing. Trimmed before filtering; omit the field to search all authors rather than passing a blank string.`),
+            .describe(`Filter results to a specific author — useful for finding a user's posts on a topic. Matches the HN username exactly and is case-sensitive, so a re-cased name finds nothing. Trimmed, and one leading @ is dropped, so "@pg" searches pg; a name that still holds whitespace is rejected as a display name. Omit the field to search all authors rather than passing a blank string.`),
         storyId: z
             .number()
             .int()
@@ -592,12 +596,18 @@ export const searchHn = tool('hn_search_content', {
             .min(0)
             .optional()
             .describe('Minimum score. Applies to stories and polls, including the ask_hn, show_hn, and front_page subsets. Comments and jobs carry no points in the search index, so any minPoints excludes them — combining it with tags "comment" or "job" is rejected.'),
+        minComments: z
+            .number()
+            .int()
+            .min(0)
+            .optional()
+            .describe('Minimum comment count. Applies to stories and polls, including the ask_hn, show_hn, and front_page subsets. Any minComments, 0 included, excludes comments, jobs, and poll options, which carry no comment count in the search index, so an untagged search returns only stories and polls — combining it with tags "comment" or "job" is rejected.'),
         count: z
             .number()
             .int()
             .min(1)
             .max(COUNT_CEILING)
-            .default(30)
+            .default(COUNT_DEFAULT)
             .describe(`Number of results to return, at most ${COUNT_CAP} per call. A larger value is served at ${COUNT_CAP}: the search runs at count ${COUNT_CAP} from the hit where the requested page starts, and the notice names the count and page to pass next. With page above 0, that hit (page × count) must begin a page of ${COUNT_CAP} hits.`),
         page: z
             .number()
@@ -608,7 +618,7 @@ export const searchHn = tool('hn_search_content', {
         view: z
             .enum(['full', 'compact'])
             .default('full')
-            .describe(`How much of each hit to return. "full" includes every field, within a ${RESPONSE_BUDGET_TEXT}-byte budget on each response surface: a page over it drops highlights.text from every hit (highlightsTextOmitted), and if still over, cuts the longest text bodies to one shared length (textTruncated) — every hit stays on the page. "compact" omits the two body-text fields — \`text\` and \`highlights.text\` — which together can repeat a long comment twice per hit; everything else (id, title, url, domain, author, points, comment count, timestamp, parent story, title highlight, matchedWords) is unchanged. Use "compact" to scan many results, then pass a hit id to hn_get_thread to read the body you skipped.`),
+            .describe('How much of each hit to return. "full" includes every field. "compact" omits the two body-text fields — `text` and `highlights.text` — which together can repeat a long comment twice per hit; everything else (id, title, url, domain, author, points, comment count, timestamp, parent story, parentId, storyUrl, title highlight, matchedWords) is unchanged. Use "compact" to scan many results, then pass a hit id to hn_get_thread to read the body you skipped.'),
     }),
     output: searchOutput,
     enrichment: {
@@ -639,16 +649,14 @@ export const searchHn = tool('hn_search_content', {
             .number()
             .optional()
             .describe(`The count value served: as passed, or ${COUNT_CAP} when count was above that cap.`),
-        highlightsTextOmitted: z
-            .boolean()
-            .optional()
-            .describe(`True when highlights.text was dropped from every hit to keep this view "full" page within the ${RESPONSE_BUDGET_TEXT}-byte response budget; highlights.title and matchedWords remain. Absent otherwise.`),
         notice: z
             .string()
             .optional()
-            .describe(`Agent guidance, one string. It opens with the page state: the next page to request while more pages remain, the 1,000-hit ceiling on the last page of a search that matches more, the last valid page when the requested page is past the end, or the filters to relax when a first page comes back empty. Notes on the inputs follow it, then, on a view "full" page fitted to the ${RESPONSE_BUDGET_TEXT}-byte response budget, a sentence naming what was omitted and the ids of hits whose text was cut. Absent when none applies.`),
+            .describe('Agent guidance, one string. It opens with the page state: the next page to request while more pages remain, the 1,000-hit ceiling on the last page of a search that matches more, the last valid page when the requested page is past the end, or the filters to relax when a first page comes back empty. Notes on the inputs follow it. Absent when none applies.'),
+    },
+    enrichmentTrailer: {
+        criteria: { render: renderCriteria },
     },
-    enrichmentTrailer: searchTrailer,
     async handler(input, ctx) {
         const { dateRange } = input;
         if (dateRange) {
@@ -663,6 +671,9 @@ export const searchHn = tool('hn_search_content', {
         if (input.minPoints != null && (input.tags === 'comment' || input.tags === 'job')) {
             throw ctx.fail('min_points_unscored_type', `minPoints cannot match tags "${input.tags}": ${input.tags} records carry no points, so any minPoints excludes every one.`);
         }
+        if (input.minComments != null && (input.tags === 'comment' || input.tags === 'job')) {
+            throw ctx.fail('min_comments_unscored_type', `minComments cannot match tags "${input.tags}": ${input.tags} records carry no comment count, so any minComments excludes every one.`);
+        }
         const filters = appliedFilters(input);
         if (input.query == null && filters.length === 0) {
             throw ctx.fail('missing_query_or_filter', 'No query and no filter were supplied, so there is nothing to search by.');
@@ -681,8 +692,7 @@ export const searchHn = tool('hn_search_content', {
         };
         const hn = getHnService();
         const result = await hn.search(params, ctx);
-        const includeBody = input.view === 'full';
-        const hits = result.hits.map((hit) => projectHit(hit, includeBody, includeBody));
+        const hits = result.hits.map((hit) => projectHit(hit, input.view === 'full'));
         ctx.log.info('Search completed', {
             query: params.query,
             hits: hits.length,
@@ -696,12 +706,13 @@ export const searchHn = tool('hn_search_content', {
             ? undefined
             : (result.exhaustive?.nbHits ?? result.exhaustiveNbHits);
         const bounds = params.dateRange ? dateRangeToEpochSeconds(params.dateRange) : {};
-        const pageFields = {
+        ctx.enrich({
             criteria: {
                 ...(params.tags != null && { tags: params.tags }),
                 ...(params.author != null && { author: params.author }),
                 ...(params.storyId != null && { storyId: params.storyId }),
                 ...(params.minPoints != null && { minPoints: params.minPoints }),
+                ...(params.minComments != null && { minComments: params.minComments }),
                 ...((bounds.start != null || bounds.end != null) && {
                     dateRange: {
                         ...(bounds.start != null && { start: isoSeconds(bounds.start) }),
@@ -717,9 +728,10 @@ export const searchHn = tool('hn_search_content', {
             page,
             totalPages,
             ...(totalHits > HIT_CEILING && { reachableHits: HIT_CEILING }),
-        };
-        const morePages = shown > 0 && page + 1 < totalPages;
-        const noticeFacts = {
+        });
+        if (shown > 0 && page + 1 < totalPages)
+            ctx.enrich.truncated({ shown, cap: params.count });
+        const notice = composeSearchNotice({
             ...params,
             droppedPrefix: searched?.droppedPrefix,
             shown,
@@ -728,61 +740,10 @@ export const searchHn = tool('hn_search_content', {
             totalHitsExact,
             totalPages,
             overCap,
-        };
-        /** The response serving `served`, with the notice and flag for how `budget` fitted it. */
-        const respond = (served, budget) => ({
-            output: { hits: served, ...(params.query != null && { query: params.query }) },
-            notice: composeSearchNotice({ ...noticeFacts, budget }),
-            highlightsTextOmitted: budget?.highlightsTextOmitted === true,
-        });
-        /** Both surfaces of a response as the caller receives it, enrichment and trailer included. */
-        const measure = (response) => measureResponse({
-            content: formatSearch(response.output),
-            result: response.output,
-            enrichment: {
-                ...pageFields,
-                ...(response.highlightsTextOmitted && { highlightsTextOmitted: true }),
-                ...(morePages && { truncated: true, shown, cap: params.count }),
-                ...(response.notice != null && { notice: response.notice }),
-            },
-            trailer: searchTrailer,
         });
-        let response = respond(hits);
-        if (includeBody && !fitsBudget(measure(response))) {
-            /** Step 1: drop the body highlight, which repeats the matched body in full, from every hit. */
-            const highlightsTextOmitted = hits.some((hit) => hit.highlights?.text != null);
-            const lean = highlightsTextOmitted
-                ? result.hits.map((hit) => projectHit(hit, true, false))
-                : hits;
-            if (highlightsTextOmitted)
-                response = respond(lean, { highlightsTextOmitted, cutIds: [] });
-            if (!highlightsTextOmitted || !fitsBudget(measure(response))) {
-                /** Step 2: cut every body longer than the largest limit they can share. */
-                const bodies = lean.map((hit) => ({ hit, bytes: hit.text ? byteLength(hit.text) : 0 }));
-                const cutAt = (limit) => {
-                    const served = bodies.map(({ hit, bytes }) => hit.text && bytes > limit
-                        ? { ...hit, text: cutToBytes(hit.text, limit), textTruncated: true }
-                        : hit);
-                    const cutIds = served.filter((hit) => hit.textTruncated).map((hit) => hit.id);
-                    return respond(served, { highlightsTextOmitted, cutIds });
-                };
-                const limit = largestFittingLimit(bodies.map((body) => body.bytes), (limitBytes) => measure(cutAt(limitBytes)));
-                response = cutAt(limit);
-                ctx.log.info('Fitted search page to the response budget', {
-                    limitBytes: limit,
-                    cutIds: response.output.hits.filter((hit) => hit.textTruncated).map((hit) => hit.id),
-                });
-            }
-        }
-        ctx.enrich({
-            ...pageFields,
-            ...(response.highlightsTextOmitted && { highlightsTextOmitted: true }),
-        });
-        if (morePages)
-            ctx.enrich.truncated({ shown, cap: params.count });
-        if (response.notice)
-            ctx.enrich.notice(response.notice);
-        return response.output;
+        if (notice)
+            ctx.enrich.notice(notice);
+        return { hits, ...(params.query != null && { query: params.query }) };
     },
     format: formatSearch,
 });