feat!: adopt mod-utils v6 (mod-llm v2)#105
Merged
Merged
Conversation
Bump the module's own Go path to github.com/gethinode/mod-llm/v2 and the mod-utils requirement to v6 v6.0.1, updating go.mod, config.toml, exampleSite/hugo.toml, README.md, and exampleSite/content/docs/getting-started.md. mod-llm has zero InitArgs.html/InitTypes.html call sites: it has no shortcodes or partials that validate arguments. Its llms-components.json generator reads site.Data.structures (the raw _arguments.yml/_types.yml schema files) directly, never through the Args API, so there is no argument-handling call site to migrate in this PR. hugo mod tidy (exampleSite) drops the now-stale mod-llm v1.3.5 and mod-utils/v5 requires. It also prunes a pre-existing unused mod-mermaid/v3 entry, confirmed via a pristine-tree tidy run to be unrelated to this change. BREAKING CHANGE: requires github.com/gethinode/mod-llm/v2 import path; adopts mod-utils v6 (no InitArgs/InitTypes call sites existed to migrate to the Args API — see gethinode/mod-utils v5-to-v6 migration notes). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
Author
|
🎉 This PR is included in version 2.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
github.com/gethinode/mod-llm/v2and themod-utilsrequirement tov6 v6.0.1(neverv6.0.0), acrossgo.mod,config.toml,exampleSite/hugo.toml,README.md, andexampleSite/content/docs/getting-started.md. Part of the mod-utils v6adoption wave: feat!: redesign argument and type system as v6 (core + shims, golden-tested) mod-utils#334 (see the v5-to-v6 migration notes in
mod-utils' README).
InitArgs.html/InitTypes.html(orArgs.html) call sites — it has noshortcodes or partials that validate arguments. Its
llms-components.jsongenerator (
layouts/index.llmscomponents.json) readssite.Data.structures(the raw
_arguments.yml/_types.ymlschema files mod-utils mounts)directly, never through the Args API, so this PR is a mechanical path bump
plus dependency cleanup, not an Args-API migration.
hugo mod tidy(run fromexampleSite/, which uses Hugo workspace wiring —mod-llm.work+use ../) drops the now-stalemod-llm v1.3.5andmod-utils/v5 v5.23.4requires fromexampleSite/go.mod/go.sum. It alsoprunes a pre-existing unused
mod-mermaid/v3entry — confirmed via apristine-tree
hugo mod tidyrun to be unrelated to this change (alreadyprunable before this PR).
Verification
hugo mod graph(fromexampleSite/):github.com/gethinode/mod-llm/v2→github.com/gethinode/mod-utils/v6@v6.0.1+vendor. Zeromod-utils/v5edgesanywhere in the graph.
pnpm build(hugo --gc --minify -s exampleSite): exit 0, 0ERRORlines,both before and after this change.
mod-fontawesome pilot, feat!: adopt mod-utils v6 (mod-fontawesome v6) mod-fontawesome#329): this module's
exampleSiteis unusually rich for wave 1 — it mounts the full Hinode +mod-blocks + mod-docs + mod-bootstrap-icons stack (via a Hugo workspace) so
its own
llms.txt/markdown/JSON generators have real content to crawl.Because this module's
go.modnow requiresmod-utils/v6, it wins theshared
layouts/partials/utilities/mount for the wholeexampleSite,flipping every other mounted module's
InitArgs/Argscalls onto the v6engine too (those modules are still individually pinned to v5, unreleased
on v6). This surfaces ~60 new
WARNlines and reflows ~12 screenshot pages,entirely from other modules' partials (
assets/hero-image.html,assets/table.html,assets/sidebar.html, mod-blocks component wrappers,etc.) — not from mod-llm's own code, which has no call sites. Four of the
affected pages (
table,navs-and-tabs,video,file) reproduce theexact mechanism the program register already documented and accepted from
the pilot's engine-endstate run ("legacy
InitTypesderived UDT doc-tablesfrom the GLOBAL type, ignoring the structure's inline type override — v6
honors the override"); the rest are the same "nested defaults now apply
correctly" / "unsupported nested attribute now reported" behavior the
mod-utils v5-to-v6 migration notes document. None are fixable in this repo
(different modules own that code) and none are ERROR lines.
distinctive output (not just HTML, since it mostly emits llms.txt-style
text) between a pristine pre-migration build and this branch's build:
llms.txtis byte-identical; every generated*/index.mdpage isbyte-identical except the one file this PR intentionally edited
(
docs/getting-started/index.md, the install-path snippet); andllms-components.jsonis structurally identical once the inherentlyvolatile
meta.generatedbuild timestamp is normalized (all 60 components,all types, byte-for-byte).
node tests/visual/visual.mjs) run against this module'sown
exampleSite, baseline shot from the pristine pre-migration clonebefore any edit: 61/73 pages pixel-clean (
0.0000%–0.08%,underthreshold), 12 flagged — all traced to the engine-endstate mechanism above
via the build's own WARN output, none are unexplained.
Test plan
pnpm buildexits 0 with 0ERRORlines (before and after)hugo mod graph(exampleSite context) showsmod-utils/v6 v6.0.1,zero
mod-utils/v5edgesInitArgs/InitTypes/Args.htmlcall sites exist inthis module (nothing to migrate)
llms.txt/*/index.md/llms-components.jsonbyte-diffed againstthe pristine pre-migration build (see Verification above)
baseline captured before any change; all 12 flagged pages traced to a
named, already-documented v6 engine-endstate mechanism in other
modules' code
mod-llm(bare path) /mod-utils/v5references: zero hits after this change
(
git show -s --format=%B HEAD | grep -E '^-.*-$'): no match🤖 Generated with Claude Code