deploy: fix update-release release-root metadata path leak#130
Merged
Conversation
Reduce control-agent SKILL.md from 476 to 303 lines (36%) by: - Deduplicating sentry agent section - Extracting Slack integration details to SLACK.md (progressive disclosure) - Removing health checks section (duplicates HEARTBEAT.md) - Trimming explanations Claude already knows - Replacing placeholder repo names with dynamic lookup Trim dev-agent (225->186) and sentry-agent (121->113) similarly.
- Re-add SLACK_BOT_TOKEN to sentry-agent env vars (required by list action) - Restore SECURITY NOTICE preamble in SLACK.md message example
Pi agents load SKILL.md as a single document and cannot follow markdown links to separate files. The progressive disclosure pattern doesn't apply here — skills must be self-contained.
Greptile SummaryFixed a path leak in
The SKILL.md files were condensed to reduce token usage in agent prompts while preserving all operational guidance. Confidence Score: 5/5
Important Files Changed
Last reviewed commit: 7719186 |
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
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.
Summary
bin/update-release.shso release paths are normalized after env/CLI parsingBAUDBOT_RELEASE_ROOTwins over inheritedBAUDBOT_SOURCE_*variablesBAUDBOT_SOURCE_URL_FILE/BAUDBOT_SOURCE_BRANCH_FILEat point of usebin/update-release.test.shfor stale source-path env varsValidation
bin/update-release.test.shbin/rollback-release.test.shnpm run test:shellnpm run lint:shell(fails locally: shellcheck not installed in PATH)