Repository navigation
AGENTS.md + CLAUDE.md: fix stale facts, tune for current agents - #1203
Merged
Merged
Conversation
- Image path: served copies live in public/<section>/, not next to the page
- Frontmatter needs description too (matches CLAUDE.md)
- Add redirect_from usage, the MDX {braces} trap, public-comment warning
- Verify section now mentions check:cli, which CI runs on every PR
- List all four integrations; drop the triple-stated sidebar rule
- Replace caps/bold emphasis with reasons, for literal-reading models
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- fact-checker / guide-tester / migration-placement live in the gitignored .claude/; say so and what to do when they're missing - scripts/shoot.mjs is gitignored (the file said it wasn't) - Dev server: don't pkill every astro dev, other worktrees run their own - Drop rules now covered by AGENTS.md (frontmatter) and the note about guardrails that no longer apply; point redirects at redirect_from - Replace caps/bold emphasis with reasons, for literal-reading models Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Someone is attempting to deploy a commit to the Keboola Engineering Team on Vercel. A member of the Team first needs to authorize it. |
keboola-pr-reviewer-bot
approved these changes
Oct 14, 2026
keboola-pr-reviewer-bot
left a comment
keboola-pr-reviewer-bot
left a comment
There was a problem hiding this comment.
Verdict: auto_approve (risk 1/5) · profile connection-docs
Auto-approve: prose-only cleanup of the repo-root contributor guides AGENTS.md and CLAUDE.md, with no public-site or build impact.
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
2 participants
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.
Jira issue(s): none
Changes:
public/<section>/(the old advice to put them next to the Markdown gives a 404). Frontmatter now asks fortitle,sluganddescription, which matches CLAUDE.md.redirect_fromfor moved or deleted pages, the MDX{braces}trap, and comments reaching the public Markdown twin. The verify list now includesnpm run check:cli(CI runs it on every PR) andnpm test.keboola-docsplugin's, with a fallback when the plugin isn't installed. Before this, a fresh clone pointed agents at tools that don't exist.scripts/shoot.mjsis gitignored (the file said it wasn't). The dev-server rule no longer sayspkill -f "astro dev", which also kills the servers of other worktrees.The feedback-widget lines from #1003 are kept as merged.
Docs-only change to the contributor guides; no page or build output changes.
🤖 Generated with Claude Code