iit-mcp-hudu 0.4.0 → 0.5.0

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## Unreleased
+
+## 0.5.0
+
+**Upgrading from 0.4.0:** clients now see `markdown_lossy` / `markdown_lossy_codes` on `articles.getSection`, with a section-specific note (the signal describes the whole article, so the loss may be in another section). Whole-article read text is unchanged. `hudu_list` compact rows are unchanged. node-hudu 0.17.0 -> 0.18.0, plan hash re-stamped to `9d110798...`, operation set unchanged (236).
+
+- `articles.getSection` lifts the SDK's whole-body `content_loss` into `markdown_lossy` / `markdown_lossy_codes` like the other article reads; `content_loss` is never passed through. This supersedes the 0.4.0 note that section reads carry no signal (#42, #39).
+- `COMPACT_FIELDS` is built from the SDK's `SUMMARY_FIELDS` plus explicit extras (`created_at`, `url`), with a test that derives from the SDK and a literal pin of the final lists, so SDK drift fails CI. Served rows are byte-identical (#42, #39).
+- Dependency: node-hudu 0.17.0 -> 0.18.0 (exact pin). Plan hash re-stamped to `9d110798...`; operation set unchanged (236).
+
 ## 0.4.0
 
 **Upgrading from 0.3.0:** `hudu_list` now returns compact summary rows by default. Callers that read custom `fields`, article `content` or layout `fields` from list rows must pass `expand: true`. Everything else is additive or a fix.

package/README.md

@@ -31,7 +31,7 @@ four old names. Rows are compact summaries by default (id, name, key scalar fiel
 25-row assets page under 20,000 characters instead of ~88,000; pass `expand: true` for the full
 records (`hudu_read` `articles.list` rows are always full). Article rows carry no body unless expanded. Paging is unchanged (`asset_layouts` still takes no `page_size`).
 
-These tools run on node-hudu 0.17.0; the `SDK behaviour (0.12.0)` section below covers the behaviours that shape them: redirect refusal, the closed input contract, the error envelope, and the client opt-ins. Numeric ids must be positive integers on every tool; anything else is refused before a request is made.
+These tools run on node-hudu 0.18.0; the `SDK behaviour (0.12.0)` section below covers the behaviours that shape them: redirect refusal, the closed input contract, the error envelope, and the client opt-ins. Numeric ids must be positive integers on every tool; anything else is refused before a request is made.
 
 `hudu_fetch_many` (node-hudu 0.12.0) fetches up to 20 single records across mixed resources in
 one call: each item is exactly `{ resource, id, fields? }` — one of the 22 bare-id resources, a
@@ -281,10 +281,10 @@ those operations take no format. Rules a caller needs:
   (for example a `<video>`); send HTML to keep it. The SDK's loss guard runs on every write.
 - Prefer Markdown (or editor-native HTML): Hudu reshapes API-written HTML on the next UI save (h5/h6 flattened, `small`/`abbr`/`q`/`section`/`aside` unwrapped, `thead` removed, an unknown callout type becomes info).
 - `articles.getSection` / `articles.patchSection` read or edit one section as Markdown, with dry-run.
-  `articles.getSection` carries no `markdown_lossy` signal (the SDK drops `content_loss` inside it, and a
-  section is not the whole body); read the whole article with `articles.get` to learn whether it is lossy.
+  `articles.getSection` carries the same `markdown_lossy` / `markdown_lossy_codes` signal as `articles.get`;
+  it describes the whole article, so the loss may be in another section.
 - A Markdown read can be incomplete: anything the dialect cannot express is dropped or reduced
-  (`<video>` and `<script>` are dropped entirely). `hudu_read` (`articles.get` / `getContext` / `resolve`) and
+  (`<video>` and `<script>` are dropped entirely). `hudu_read` (`articles.get` / `getContext` / `getSection` / `resolve`) and
   `hudu_get_article_context` then set `markdown_lossy: true` (with `markdown_lossy_codes`, the SDK's
   `diffArticleRoundTrip` content-loss codes) and add one sentence to the result text; fetch the stored
   HTML with `hudu_fetch_many` (`{items:[{resource:"articles", id}]}`) to see the full body. The read is one
@@ -371,7 +371,7 @@ pnpm run dev
 
 CI (`.github/workflows/ci.yml`) runs `typecheck`, `lint` and `test` on every push and pull request.
 
-`node-hudu` is pinned to `0.17.0`. The catalog hash test detects metadata drift; upgrades require
+`node-hudu` is pinned to `0.18.0`. The catalog hash test detects metadata drift; upgrades require
 review of the client authority, operation enums and redaction regressions. Docker uses `pnpm install --frozen-lockfile`.
 
 ## Deployment

package/dist/tools/article-loss.js

@@ -14,6 +14,8 @@ const RECORD_PATHS = {
     // resolve answers the record itself; the `{ value }` envelope is the helper form of the same read.
     'articles.resolve': [[], ['value']],
     'articles.getContext': [['article']],
+    // The SDK reports getSection loss for the WHOLE body (the loss may sit in another section).
+    'articles.getSection': [[]],
 };
 const isObject = (v) => v !== null && typeof v === 'object' && !Array.isArray(v);
 const take = (node, path, codes) => {
@@ -43,4 +45,4 @@ export function takeContentLoss(operation, result) {
     return { result: stripped, codes: [...codes].sort() };
 }
 /** The one short sentence appended to the result text when a read lost content. */
-export const lossyNote = (codes) => ` markdown_lossy: true (${codes.join(', ')}): the Markdown body is incomplete; fetch the stored HTML with hudu_fetch_many {items:[{resource:"articles", id}]} to see all of it.`;
+export const lossyNote = (codes, operation) => ` markdown_lossy: true (${codes.join(', ')}): ${operation === 'articles.getSection' ? 'the Markdown read of this article is incomplete (the loss may be in another section)' : 'the Markdown body is incomplete'}; fetch the stored HTML with hudu_fetch_many {items:[{resource:"articles", id}]} to see all of it.`;

package/dist/tools/dispatch.js

@@ -120,7 +120,7 @@ export function registerDispatchTools(server, ctx) {
                 }
                 const simulated = result !== null && typeof result === 'object' && result.simulated === true;
                 // A Markdown article read that lost content says so; a lossless read is byte-identical to before.
-                return ok((simulated ? `Dry run: nothing was written (${operation}).` : `${operation} completed.`) + (lossy.length ? lossyNote(lossy) : ''), {
+                return ok((simulated ? `Dry run: nothing was written (${operation}).` : `${operation} completed.`) + (lossy.length ? lossyNote(lossy, operation) : ''), {
                     simulated,
                     operation,
                     result: result ?? null,

package/dist/tools/lists.js

@@ -1,4 +1,5 @@
 import * as z from 'zod/v4';
+import { SUMMARY_FIELDS } from 'node-hudu';
 import { getCapability } from 'node-hudu/capabilities';
 import { configError } from 'node-hudu/mcp';
 import { assertSecretReadPermitted, assertWritePermitted } from '../policy.js';
@@ -96,14 +97,28 @@ function assertNonSearchableListReadPermitted(operation, ctx) {
     assertSecretReadPermitted(operation, config, log);
 }
 /**
- * Compact row fields per resource: the node-hudu `*Summary` shapes plus `created_at`. The SDK exports no
- * runtime projection, so `test/tools/lists-compact.test.ts` only pins these lists to a literal copy: it forces a reviewer to revisit them on a node-hudu bump, it does not detect SDK drift itself. `hudu_list` projects each row onto these unless `expand: true`.
+ * Extras beyond the SDK's `SUMMARY_FIELDS` (the keys of the node-hudu `*Summary` shapes). Timestamps go last
+ * in the served row, so `updated_at` (an SDK key where present) is moved after these extras.
+ */
+const COMPACT_EXTRAS = {
+    companies: ['created_at'],
+    articles: ['url', 'created_at'],
+    assets: ['created_at'],
+    asset_layouts: ['url', 'created_at', 'updated_at'],
+};
+const compactFields = (r) => {
+    const base = SUMMARY_FIELDS[r];
+    return [...base.filter((k) => k !== 'updated_at'), ...COMPACT_EXTRAS[r], ...(base.includes('updated_at') ? ['updated_at'] : [])];
+};
+/**
+ * Compact row fields per resource: `SUMMARY_FIELDS` plus `COMPACT_EXTRAS`. `test/tools/lists-compact.test.ts`
+ * derives them from the SDK and also pins the final lists literally. `hudu_list` projects each row onto these unless `expand: true`.
  */
 export const COMPACT_FIELDS = {
-    companies: ['id', 'name', 'nickname', 'slug', 'website', 'phone_number', 'city', 'state', 'id_number', 'archived', 'url', 'created_at', 'updated_at'],
-    articles: ['id', 'name', 'slug', 'company_id', 'folder_id', 'draft', 'enable_sharing', 'url', 'created_at', 'updated_at'],
-    assets: ['id', 'name', 'company_id', 'company_name', 'asset_layout_id', 'primary_serial', 'asset_type', 'archived', 'url', 'created_at', 'updated_at'],
-    asset_layouts: ['id', 'name', 'slug', 'active', 'icon', 'color', 'url', 'created_at', 'updated_at'],
+    companies: compactFields('companies'),
+    articles: compactFields('articles'),
+    assets: compactFields('assets'),
+    asset_layouts: compactFields('asset_layouts'),
 };
 /** Project rows onto the compact fields; a field the row lacks is omitted, not nulled. */
 export function compactRows(resource, rows) {

package/dist/tools.js

@@ -30,4 +30,4 @@ export { MAX_SCOPED_CLIENTS, ScopedClientCache, createMcpServerFactory } from '.
  * it against the installed `CATALOG_PLAN_HASH`, so the upgrade fails CI instead of quietly shipping
  * a tool surface that no longer describes the SDK behind it.
  */
-export const BUILT_AGAINST_PLAN_HASH = '1ad5e23b9c140e8f9cf60ac710dd508fc1109e86b9a0620b2d0925f16b11d7b8';
+export const BUILT_AGAINST_PLAN_HASH = '9d11079847a9e2a8d73f4a075a4004b49f59624be5ffc8253636fc41cf677702';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "iit-mcp-hudu",
-  "version": "0.4.0",
+  "version": "0.5.0",
   "description": "MCP server for Hudu IT documentation, built on the node-hudu SDK",
   "license": "MIT",
   "author": "Max Soukhomlinov",
@@ -43,7 +43,7 @@
   },
   "dependencies": {
     "@modelcontextprotocol/server": "^2.3.1",
-    "node-hudu": "0.17.0",
+    "node-hudu": "0.18.0",
     "zod": "^4.6.7"
   },
   "devDependencies": {