chore(dev→main): routine PR cadence for CodeRabbit review

[Snider]

Routine dev→main PR opened by Hephaestus PR-cadence task.

This PR exists to:

1. Trigger CodeRabbit auto-review of accumulated dev work
2. Surface any blocking review feedback before merge
3. Keep main current with dev once approved

ahead_by: 55 commit(s) (per gh api compare)

If CodeRabbit clears with no blocking comments, this PR is eligible for gh pr merge --merge (real merge commit, no force-push). Conflicts and review feedback should be addressed on the dev branch before merge.

Co-authored-by: Hephaestus hephaestus@cladius

[coderabbitai]

Warning

Rate limit exceeded

@Snider has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 7 minutes and 36 seconds before requesting another review.

To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 03c8d414-1482-4aaa-a81f-ba76e4026e91

📥 Commits

Reviewing files that changed from the base of the PR and between 5a6b0aa and e9ec57c.

⛔ Files ignored due to path filters (3)

• go.sum is excluded by !**/*.sum
• go.work is excluded by !**/*.work
• go.work.sum is excluded by !**/*.sum

📒 Files selected for processing (55)

• .core/build.yaml
• .core/release.yaml
• .gitignore
• README.md
• SESSION-BRIEF.md
• Taskfile.yaml
• bench_test.go
• cmd/codegen/main.go
• cmd/codegen/main_test.go
• cmd/wasm/main.go
• cmd/wasm/main_test.go
• cmd/wasm/register.go
• cmd/wasm/register_test.go
• cmd/wasm/render_layout.go
• cmd/wasm/render_layout_test.go
• cmd/wasm/size_test.go
• codegen/codegen.go
• codegen/codegen_test.go
• codegen/typescript.go
• context.go
• context_test.go
• docs/architecture.md
• docs/history.md
• docs/index.md
• edge_test.go
• entitled.go
• entitled_test.go
• go.mod
• grammar.go
• grammar_test.go
• integration_test.go
• layout.go
• layout_test.go
• node.go
• node_test.go
• path.go
• path_test.go
• pipeline.go
• pipeline_test.go
• pkg/api/handlers.go
• pkg/api/handlers_test.go
• pkg/api/provider.go
• pkg/api/provider_test.go
• render.go
• render_test.go
• responsive.go
• responsive_test.go
• shadow.go
• shadow_test.go
• tests/cli/html/Taskfile.yaml
• text_translate.go
• text_translate_args.go
• text_translate_default.go
• wasm.go
• wasm_test.go

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Snider]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[Copilot]

Pull request overview

Routine dev→main sync PR that consolidates recent go-html changes, including module/dep migration, new WASM and API surfaces, and a revised deterministic data-block path scheme.

Changes:

• Migrates the module/import paths to the dappco.re/go/* family and updates consumers/tests/docs accordingly.
• Revises layout/responsive rendering to use dotted data-block coordinates (and updates parsing/tests) plus adds responsive media hints.
• Adds new surfaces: WASM globalThis bridge (coreHTML.renderToString), a Gin-based API provider, and expanded web-component codegen (incl. TS defs).

Reviewed changes

Copilot reviewed 56 out of 58 changed files in this pull request and generated 18 comments.

Show a summary per file

File	Description
wasm_test.go	Adds WASM-side tests for the globalThis.coreHTML.renderToString bridge.
wasm.go	Introduces a WASM JS bridge that parses JS values into Nodes and renders to string.
text_translate_default.go	Switches default translation import to the new i18n module path.
text_translate_args.go	Adds count-argument normalization/injection for i18n.count.* keys.
text_translate.go	Injects normalized translation args before delegating to translator/default.
tests/cli/html/Taskfile.yaml	Adds CLI task runner for build/vet/test in tests.
shadow_test.go	Adds tests for Web Component JS output and naming helpers.
shadow.go	Introduces ShadowComponent JS generation + JS string literal escaping helpers.
responsive_test.go	Updates tests for dotted block IDs and adds media-hint test coverage.
responsive.go	Adds Responsive.Add(..., media) and preserves layout-path rendering through responsive.
render_test.go	Updates i18n import path in render tests.
render.go	Adds WASM-size budget note; keeps Render helper dependency-free.
pkg/api/provider_test.go	Adds tests for the new provider metadata and base path.
pkg/api/provider.go	Adds a Gin provider wrapper exposing /render and /grammar/check + route metadata.
pkg/api/handlers_test.go	Adds handler tests for render and grammar endpoints.
pkg/api/handlers.go	Implements /render and /grammar/check handlers.
pipeline_test.go	Adds StripTags regression tests and deterministic CompareVariants key ordering test.
pipeline.go	Improves StripTags correctness and makes CompareVariants deterministic + context cloning.
path_test.go	Updates/expands tests for dotted block IDs and compatibility parsing.
path.go	Implements dotted + legacy-hyphenated ParseBlockID parsing with validation.
node_test.go	Adds coverage for new block-path propagation and Attr recursion through wrappers.
node.go	Adds layout-path rendering plumbing, typed-nil handling, and Attr recursion into Layout/Responsive.
layout_test.go	Updates slot semantics and data-block expectations to dotted coordinates.
layout.go	Changes slot semantics (L→nav), dotted data-block generation, and layout-path cloning renderer.
integration_test.go	Updates i18n import path in integration tests.
grammar_test.go	Adds determinism/path-budget tests for GrammarImprint.
grammar.go	Introduces structural GrammarImprint stamping without rendering content.
go.work.sum	Adds workspace sum entries for new dependency graph.
go.work	Adds Go workspace file.
go.sum	Updates sums to match dependency/module changes.
go.mod	Renames module to dappco.re/go/html, updates direct deps, and adds local replaces.
entitled_test.go	Adds tests for new Entitled checker overload behavior.
entitled.go	Adds EntitlementChecker interface and overload-style Entitled helper w/ empty sentinel.
edge_test.go	Updates deep nesting/path expectations and variant-validation expectations.
docs/index.md	Updates docs for new features/paths (but currently has module path mismatch vs go.mod).
docs/history.md	Updates project history notes (variant validation + TS defs).
docs/architecture.md	Updates architecture docs for new path format and Responsive media hints.
context_test.go	Adds tests for locale forwarding, metadata aliasing, and cloneContext behavior.
context.go	Adds Data/Metadata aliasing, locale selection logic, and cloneContext helpers.
codegen/typescript.go	Escapes tags when emitting TS HTMLElementTagNameMap entries.
codegen/codegen_test.go	Updates tests away from testify and expands tag/classname cases.
codegen/codegen.go	Adds reserved-name checks, JS string escaping, and adjusts tag→classname logic.
cmd/wasm/size_test.go	Updates imports and assertions; keeps WASM size gate test.
cmd/wasm/render_layout_test.go	Adds test ensuring empty slot strings still render containers.
cmd/wasm/render_layout.go	Adds shared legacy layout rendering helper.
cmd/wasm/register_test.go	Updates tests away from testify for the WASM component registration path.
cmd/wasm/register.go	Updates imports to new module paths for codegen/log.
cmd/wasm/main_test.go	Updates expected output for new data-block format and adds empty-slot test.
cmd/wasm/main.go	Changes WASM export behavior and adds template-string rendering with scalar substitution.
cmd/codegen/main_test.go	Updates tests away from testify and validates watch-mode behavior.
cmd/codegen/main.go	Refactors CLI to use core’s command system; adds generate/types/watch subcommands.
bench_test.go	Updates i18n import path in benchmarks.
Taskfile.yaml	Adds top-level task to build WASM output into dist/.
SESSION-BRIEF.md	Updates session brief to reflect current status/outputs.
README.md	Updates README content (but currently has module path mismatch vs go.mod).
.gitignore	Ignores dist/ output directory.
.core/release.yaml	Removes core release config.
.core/build.yaml	Removes core build config.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

SPDX header tag is misspelled. Most license scanners expect the exact SPDX-License-Identifier token; SPDX-Licence-Identifier may not be recognized.

[Copilot]

SPDX header tag is misspelled. Most license scanners expect the exact SPDX-License-Identifier token; SPDX-Licence-Identifier may not be recognized.

[Copilot]

sameMetadataMap() detects aliasing by temporarily writing a probe key into the caller-provided map. This has observable side effects and can introduce data races if the context maps are accessed concurrently. Consider tracking aliasing explicitly on Context (e.g., an unexported boolean set by NewContext/cloneContext) or using a non-mutating check (e.g., compare map header pointers via reflect/unsafe, if acceptable for WASM size).

[Copilot]

GrammarImprint treats any Each/EachSeq node as having 0 children unless it has a non-nil items slice. For EachSeq(), items is empty and seq is set, so structuralChildCount/structuralEmpty will report the node as empty even when it would render items. Consider either (1) materialising the seq for counting in imprinting, or (2) treating non-nil seq as “non-empty/branch” (possibly with a dedicated tag) so stamps don’t incorrectly collapse dynamic lists to leaf/empty.

[Copilot]

SPDX header tag is misspelled. Most license scanners expect the exact SPDX-License-Identifier token; SPDX-Licence-Identifier may not be recognized.

[Copilot]

Quick Start import uses dappco.re/go/core/html, but the module path in go.mod is dappco.re/go/html. This example won't compile as written; update the import path (and any other occurrences) to match go.mod.

[Copilot]

SPDX header tag is misspelled. Most license scanners expect the exact SPDX-License-Identifier token; SPDX-Licence-Identifier may not be recognized.

[Copilot]

SPDX header tag is misspelled. Most license scanners expect the exact SPDX-License-Identifier token; SPDX-Licence-Identifier may not be recognized.

[Copilot]

SPDX header tag is misspelled. Most license scanners expect the exact SPDX-License-Identifier token; SPDX-Licence-Identifier may not be recognized.

Suggested change

	// SPDX-Licence-Identifier: EUPL-1.2
	// SPDX-License-Identifier: EUPL-1.2

[Copilot]

These tests assert that tags like "math-α" are valid custom element names. In browsers, custom element names are restricted to a specific ASCII-only pattern; non-ASCII names typically throw at customElements.define(). If this project targets standard Web Components, update the tests (and TagToClassName expectations) to only cover spec-valid tags.