Repository navigation
fix(ai-run): reserved env var + artifact name - #33
Merged
Merged
Conversation
…tain "/" Two bugs found by dispatching ai-run.yml for real: - `GITHUB_REPOSITORY` cannot be overridden via `env:` (GitHub reserves GITHUB_*), so the runner saw the hub repo instead of the target and failed to read the issue. Add `AI_TARGET_REPO` (read first by main.ts) and set it in ai-run.yml. - the artifact name embedded `owner/repo`, which upload-artifact rejects (forward slash is not allowed) -> use issue + run id.
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.
实测 dispatch
ai-run.yml暴露的两个 bug：GITHUB_REPOSITORY是 GitHub 保留变量，env:覆盖无效 → runner 认成 hub 仓库、读不到目标 Issue。改为传AI_TARGET_REPO（main.ts 优先读它）。owner/repo含/，upload-artifact 拒绝 → 改成 issue + run id。