Skip to content

fix(plugin): trim keboola-expert.md under the prompt budget - #843

Merged
soustruh merged 1 commit into
mainfrom
fix/expert-prompt-budget
Oct 11, 2026
Merged

soustruh merged 1 commit into
mainfrom
fix/expert-prompt-budget

Conversation

@soustruh

Copy link
Copy Markdown
Contributor

What was wrong

What changed

  • The file is now 67,958 bytes. This leaves about 2,000 bytes for the open PRs. The budget is not changed.
  • Only §3 inline gotchas are shortened. No rule, version gate or safety instruction is removed. The full text of each entry is already in gotchas.md, sync-workflow.md or data-app-workflow.md, so nothing is moved and nothing is duplicated.
  • Managed-repo app with no WORKSPACE_ID: shortened to the fix and the --git-branch rule, with a pointer to data-app-workflow.md.
  • Drifted sync tree: the no-baseline CONFLICT and ENCRYPTION_FAILED notes are shorter, with a pointer to sync-workflow.md. The 0.90.1 phantom-drift history is now one sentence tied to SYNC_LEGACY_BOUNDARY.
  • Usage telemetry events: shortened to what the events are and how to turn them off.
  • --project with a numeric project ID: removed the full list of alias options and the edge cases. gotchas.md keeps them.
  • 401 error codes: shortened to the three codes that share exit 3.
  • Flow triggers: removed the past-incident story.
  • component detail storage catalog fallback: shortened to the examples_count rule.

@keboola-pr-reviewer-bot keboola-pr-reviewer-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.

Verdict: needs_human (risk 2/5) · profile _default

Escalating per policy: changes to AI/agent instruction files must never be auto-approved, even when benign.

Concerns:

  • plugins/kbagent/agents/keboola-expert.md: Agent instruction file edited — policy forbids auto-approve; verify no gotcha nuance lost

@soustruh

Copy link
Copy Markdown
Contributor Author

@keboola-pr-reviewer review override “No source code edited, just agentic instructions file. PR bot's concern verified. ദ്ദി(•ᴗ•)”

@keboola-pr-reviewer-bot

Copy link
Copy Markdown

⚠️ Override not applied. A review override needs a justification of at least 20 characters.

Usage: @keboola-pr-reviewer review override "<why this must merge without a full review>"

@soustruh

Copy link
Copy Markdown
Contributor Author

@keboola-pr-reviewer review override “No source code edited, just agentic instructions file. PR bot's concern verified.”

@keboola-pr-reviewer-bot

Copy link
Copy Markdown

⚠️ Override not applied. A review override needs a justification of at least 20 characters.

Usage: @keboola-pr-reviewer review override "<why this must merge without a full review>"

@soustruh

Copy link
Copy Markdown
Contributor Author

@keboola-pr-reviewer review override "No source code edited, just agentic instructions file. PR bot's concern verified. ദ്ദി(•ᴗ•)"

@keboola-pr-reviewer-bot

Copy link
Copy Markdown

⚠️ Override not applied. A review override needs a justification of at least 20 characters.

Usage: @keboola-pr-reviewer review override "<why this must merge without a full review>"

@soustruh

Copy link
Copy Markdown
Contributor Author

@keboola-pr-reviewer review override "No source code edited, just agentic instructions file. PR bot concern verified. ദ്ദി(•ᴗ•)"

@keboola-pr-reviewer-bot keboola-pr-reviewer-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.

🔓 Manual override approval by @soustruh

This PR was approved via break-glass override — no agent review was run.

Reason: No source code edited, just agentic instructions file. PR bot concern verified. ദ്ദി(•ᴗ•)

Logged to the central override audit stream.

@soustruh
soustruh merged commit 8b656a5 into main Oct 11, 2026
5 checks passed
@soustruh
soustruh deleted the fix/expert-prompt-budget branch October 11, 2026 23:22
@soustruh soustruh mentioned this pull request Oct 12, 2026
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.

2 participants