Skip to content

Docs: add dbt-clickhouse remote - #362

Merged
Blargian merged 6 commits into
mainfrom
codex/add-dbt-clickhouse-remote
Oct 2, 2026
Merged

Blargian merged 6 commits into
mainfrom
codex/add-dbt-clickhouse-remote

Conversation

@Blargian

@Blargian Blargian commented Oct 2, 2026

Copy link
Copy Markdown
Member

Summary

  • Add ClickHouse/dbt-clickhouse as a remote documentation source.
  • Mount its docs directory at the existing dbt integration path.

Validation

  • jq empty remotes.json
  • git diff --check

@github-actions

github-actions Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

🕵

Preview Deployment Components Updated (UTC)
Docs preview 🟢 Ready English: 🟢 Ready; translations: Production fallback 2026-10-02 13:46 UTC

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale Bugbot comment from a previous run.

Comment thread remotes.json

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit a545057. Configure here.

Comment thread bin/gen-sidebar.ts
// `{ "$ref": "./x.json" }` is replaced by the file; sibling keys (e.g.
// `{ "language": "es", "$ref": "./es/docs.json" }`) are kept on top.
const p = path.normalize(path.join(baseDir, o.$ref));
if (!fs.existsSync(p) && isInOmittedRemote(p)) return null;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Omitted refs drop sidebar entries

Medium Severity

When a navigation $ref points into an omitted remote, resolveRefs returns null and the sidebar generator drops that entry. Omitted sourceRef remotes still become a production link, so preview builds lose the dbt group instead of keeping a path to production.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit a545057. Configure here.

@Blargian
Blargian merged commit 3175f60 into main Oct 2, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant