Make test-*.md workflows self-contained by removing @include directives#32
Merged
Make test-*.md workflows self-contained by removing @include directives#32
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…tained Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] make test-*.md workflows self contained
Make test-*.md workflows self-contained by removing @include directives
Aug 13, 2025
pelikhan
approved these changes
Aug 13, 2025
| > NOTE: Include a footer link like this at the end of each new issue, issue comment or pull request you create. Do this in addition to any other footers you are instructed to include. | ||
| ### Final Step: Post Your Analysis | ||
|
|
||
| ```markdown |
Contributor
There was a problem hiding this comment.
we have lots of copies of this one
pelikhan
assigned
dsyme and
Krzysztof-Cieslak and unassigned
Krzysztof-Cieslak and
dsyme
Krzysztof-Cieslak
approved these changes
Aug 13, 2025
pelikhan
added a commit
that referenced
this pull request
Aug 25, 2025
* Replace check-team-member shared action with inlined JavaScript script (#22) * Initial plan * Replace check-team-member template with inlined JavaScript script Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Fix code formatting (remove trailing whitespace) Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Add check_team_member.cjs to tsconfig.json and create comprehensive tests Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> Co-authored-by: Peli de Halleux <pelikhan@users.noreply.github.com> * Refactor compute_text_action.yaml as embedded JavaScript with sanitization (#25) * Simplify add-reaction feature by removing fallback and try/catch logic (#24) * Initial plan * Reimplement add-reaction feature as inlined JavaScript Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Remove "remove" functionality from add-reaction feature, simplify to only support adding reactions Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Remove fallback and try/catch clauses from add-reaction feature Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> Co-authored-by: Peli de Halleux <pelikhan@users.noreply.github.com> * Remove compute-text action dependency and template files from workflow system (#29) * Fix linter issues: Remove unused reactionActionTemplate variable (#30) * Add output text to test-claude and inline JavaScript instead of shared actions (#32) * Remove obsolete reaction_action.yaml template file (#34) * Remove unnecessary content permissions and checkout steps from task and add_reaction jobs (#35) --------- Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
9 tasks
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.
This PR removes all
@includedirective dependencies from the test agentic workflows (test-codex.md,test-gemini.md, andtest-claude.md) and makes them fully self-contained.Changes Made
Removed @include dependencies:
@include agentics/shared/include-link.md- AI attribution footer template@include agentics/shared/job-summary.md- GitHub Actions job summary instructions@include agentics/shared/xpia.md- Cross-Prompt Injection Attack security guidelines@include agentics/shared/gh-extra-tools.md- GitHub CLI tools configuration@include agentics/shared/tool-refused.md- Tool access request instructionsReplaced with inline self-contained content:
Benefits
✅ Self-contained workflows - No external file dependencies
✅ Simplified deployment - Workflows can be used independently without shared includes
✅ Clear focus - Workflows now explicitly instruct posting results as PR comments
✅ Maintained functionality - All essential security and attribution content preserved
✅ Successful compilation - All test workflows compile and validate correctly
The workflows now focus on their core purpose: analyzing pull requests and posting comprehensive summaries as comments, while maintaining all necessary security protections and attribution requirements inline.
Fixes #31.
Warning
Firewall rules blocked me from connecting to one or more addresses
I tried to connect to the following addresses, but was blocked by firewall rules:
api.contoso.com/tmp/go-build642639418/b256/cli.test -test.testlogfile=/tmp/go-build642639418/b256/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true(dns block)/tmp/go-build320057617/b256/cli.test -test.testlogfile=/tmp/go-build320057617/b256/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true(dns block)/home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js(dns block)If you need me to access, download, or install something from one of these locations, you can either:
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.