-
Notifications
You must be signed in to change notification settings - Fork 103
feat(workflows): add fullsend_ai_ref for self-consistent version pinning #1278
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -23,11 +23,16 @@ on: | |
| fullsend_version: | ||
| required: false | ||
| type: string | ||
| default: 'latest' | ||
| default: "latest" | ||
| install_mode: | ||
| required: false | ||
| type: string | ||
| default: 'per-org' | ||
| default: "per-org" | ||
| fullsend_ai_ref: | ||
| description: Ref of fullsend-ai/fullsend to load actions from. Must match the ref used in the `uses:` line that calls this workflow. | ||
| type: string | ||
| required: false | ||
| default: v0 | ||
| secrets: | ||
| FULLSEND_GCP_WIF_PROVIDER: | ||
| required: true | ||
|
|
@@ -54,10 +59,14 @@ jobs: | |
| uses: actions/checkout@v6 | ||
| with: | ||
| repository: fullsend-ai/fullsend | ||
| ref: v0 | ||
| ref: ${{ inputs.fullsend_ai_ref }} | ||
| path: .defaults | ||
| fetch-depth: 1 | ||
| sparse-checkout: | | ||
| .github/actions/ | ||
| .github/scripts/ | ||
| internal/scaffold/fullsend-repo/ | ||
| action.yml | ||
|
|
||
| - name: Prepare workspace (upstream defaults + org/repo overrides) | ||
| env: | ||
|
|
@@ -92,18 +101,17 @@ jobs: | |
| done | ||
| mkdir -p .github/scripts | ||
| cp "${SRC}/.github/scripts/setup-agent-env.sh" .github/scripts/setup-agent-env.sh | ||
| rm -rf .defaults | ||
|
Member
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. HIGH — The Suggestion: Add a cleanup step with - name: Cleanup upstream checkout
if: always()
run: rm -rf .defaultsGitHub Actions resolves find .defaults -mindepth 1 -maxdepth 1 \
! -name 'action.yml' ! -name '.github' \
-exec rm -rf {} +Same pattern applies to all 5 stage workflows. Flagged by 6/7 agents.
Member
Author
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. Why should we delete the repository? It is publicly available. |
||
|
|
||
| - name: Validate enrollment and extract repo metadata | ||
| id: repo-parts | ||
| uses: fullsend-ai/fullsend/.github/actions/validate-enrollment@v0 | ||
| uses: ./.defaults/.github/actions/validate-enrollment | ||
| with: | ||
| source_repo: ${{ inputs.source_repo }} | ||
| install_mode: ${{ inputs.install_mode }} | ||
|
|
||
| - name: Mint coder token | ||
| id: app-token | ||
| uses: fullsend-ai/fullsend/.github/actions/mint-token@v0 | ||
| uses: ./.defaults/.github/actions/mint-token | ||
| with: | ||
| role: coder | ||
| repos: ${{ steps.repo-parts.outputs.name }} | ||
|
|
@@ -127,7 +135,7 @@ jobs: | |
| run: bash scripts/pre-code.sh | ||
|
|
||
| - name: Setup GCP and prepare credentials | ||
| uses: fullsend-ai/fullsend/.github/actions/setup-gcp@v0 | ||
| uses: ./.defaults/.github/actions/setup-gcp | ||
| with: | ||
| gcp_wif_provider: ${{ secrets.FULLSEND_GCP_WIF_PROVIDER }} | ||
| gcp_project_id: ${{ secrets.FULLSEND_GCP_PROJECT_ID }} | ||
|
|
@@ -143,7 +151,7 @@ jobs: | |
| run: bash .github/scripts/setup-agent-env.sh | ||
|
|
||
| - name: Run code agent | ||
| uses: fullsend-ai/fullsend@v0 | ||
| uses: ./.defaults/ | ||
| env: | ||
| GITHUB_ISSUE_URL: ${{ fromJSON(inputs.event_payload).issue.html_url }} | ||
| ISSUE_NUMBER: ${{ fromJSON(inputs.event_payload).issue.number }} | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.