Skip to content

fix: resolve MDX compilation error in using-goosehints.md#8332

Merged
baxen merged 1 commit into
mainfrom
fix/goosehints-mdx-details-tag
Apr 5, 2026
Merged

fix: resolve MDX compilation error in using-goosehints.md#8332
baxen merged 1 commit into
mainfrom
fix/goosehints-mdx-details-tag

Conversation

@baxen
Copy link
Copy Markdown
Collaborator

@baxen baxen commented Apr 5, 2026

Problem

CI fails with:

MDX compilation failed for file "documentation/docs/guides/context-engineering/using-goosehints.md"
Cause: Expected a closing tag for `<details>` (156:4-156:13) before the end of `paragraph`

Root Cause

The numbered list containing <details> tags was broken by a paragraph inserted between items 1 and 2. MDX treats the continuation as a paragraph rather than a new list item, so the <details> on item 2 is never properly closed from MDX's perspective.

Fix

  • Move the note about nested hint persistence after the numbered list and wrap it in a :::note admonition
  • Remove trailing whitespace on a </details> tag

The numbered list containing <details> tags was broken by a paragraph
inserted between items 1 and 2, causing MDX to fail with:
'Expected a closing tag for <details> before the end of paragraph'

Move the note about nested hint persistence after the numbered list
and wrap it in a :::note admonition. Also remove trailing whitespace
on a </details> tag.

Signed-off-by: Bradley Axen <baxen@squareup.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 5, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-04-05 23:46 UTC

@baxen baxen enabled auto-merge April 5, 2026 23:39
@baxen baxen disabled auto-merge April 5, 2026 23:39
@baxen baxen merged commit 85f513d into main Apr 5, 2026
24 checks passed
@baxen baxen deleted the fix/goosehints-mdx-details-tag branch April 5, 2026 23:39
lifeizhou-ap added a commit that referenced this pull request Apr 7, 2026
* origin/main: (32 commits)
  docs: rework homepage and add aaif migration blog post (#8356)
  chore(aaif): rename a bunch of repository references (#8152)
  fix: use OPENAI_API_KEY secret for recipe security scanner (#8358)
  feat: configurable extension timeouts via ACP _meta and global default (#8295)
  fix: hide hidden extensions in UI (#8346)
  refactor: skills as its own platform ext (#8244)
  fix baseUrl (#8347)
  Fix desktop slash commands (#8341)
  fix(cli): display platform-correct secrets path in keyring config dialog (#8328)
  feat(acp): add reusable ACP provider controls (#8314)
  fix: resolve MDX compilation error in using-goosehints.md (#8332)
  fix: use v1beta1 API version for Google/MaaS models on GCP Vertex AI (#8278)
  docs: add MCP Roots guide (#8252)
  rust acp client for extension methods (#8227)
  fix: reconsolidate split tool-call messages to follow OpenAI format (#7921)
  fix: clean up MCP subprocesses after abrupt parent exit (#8242)
  build: raise default stack reserve to 8 MB (#8234)
  fix(config): honour GOOSE_DISABLE_KEYRING from config.yaml at startup (#8219)
  feat: add configurable fast_model for declarative providers (#8194)
  fix(authentication): Allow connecting to Oauth servers that use protected-resource fallback instead of the WWW-authenticate header (#8148)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants