-
Notifications
You must be signed in to change notification settings - Fork 454
Mark all agentic workflows as private #40048
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,5 @@ | ||
| --- | ||
| private: true | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
💡 Suggested fixAdd a private guard in // Enforce private: true — private workflows cannot be trialled from external repos
if !fetched.IsLocal && ExtractWorkflowPrivate(string(content)) {
return fmt.Errorf("workflow '%s' is private and cannot be run in trial mode from external repositories", parsedSpec.WorkflowName)
}This mirrors the same check already performed in A test case should accompany the fix, since no existing test covers this path (the existing |
||
| on: | ||
| schedule: | ||
| - cron: daily around 10:00 | ||
|
|
||
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,5 @@ | ||
| --- | ||
| private: true | ||
| description: On-demand AIC audit for a user-specified date range | ||
| on: | ||
| workflow_dispatch: | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coverage gap: shared imports not marked private — 69 files in
.github/workflows/shared/have noprivate: truefrontmatter. SinceExtractWorkflowPrivate()returnsfalsefor them andisSupportedPackageInstallablePath()accepts any path starting with.github/workflows/(including theshared/subdirectory), an external user can rungh aw add github/gh-aw/shared/<name>and successfully copy any shared import into their repo.💡 Suggested follow-up fix
ExtractWorkflowPrivateSettingreturns(false, false)for these files because none have aprivate:YAML key — the YAML comments inside their frontmatter blocks parse as an empty map.Fix: add a follow-up PR to stamp all 69
shared/*.mdfiles withprivate: trueusing the same approach as this PR. For files with a---block:For pure-markdown imports without frontmatter (e.g.,
noop-reminder.md), add a frontmatter block:No recompile needed for shared imports — they have no
.lock.ymlcounterpart.