docs(fullsend): CI install, dispatch design, and upgrade procedures (TC-6193) - #305
Conversation
…cedures Add a CI deployment section (per-repo fullsend github setup / inference provision / org GitHub App install), a dispatch-design section (inline pull_request wait via lewagon/wait-on-check-action, CEL trigger, pre_script Jira gating), a day-2 variables/secrets management table, and maintenance procedures for bumping the pinned fullsend version and keeping the two plugin-version files in sync. Refresh stale v0.37.0 references. Implements TC-6193 Assisted-by: Claude Code Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Reviewer's GuideExpands fullsend.md with implementation-level documentation for the deployed verify-pr CI setup, event and gating design, operational variable/secret management, CLI and plugin upgrade procedures, and current Jira reporting behavior. Sequence diagram for the pull request verify-pr dispatchsequenceDiagram
participant GitHub
participant Wait as wait-on-check-action
participant Dispatch as reusable-dispatch.yml
participant OIDC
participant Vertex
participant Fullsend
participant Jira
GitHub->>Wait: pull_request opened/synchronize/reopened
Wait->>Wait: Wait for other checks to reach terminal state
Wait-->>Dispatch: Release dispatch after checks or timeout
Dispatch->>OIDC: Mint review-role App token
Dispatch->>Vertex: Obtain short-lived WIF credential
Dispatch->>Fullsend: fullsend run verify-pr
Fullsend->>Jira: pre_script JQL lookup and status/label gate
Jira-->>Fullsend: Qualifying issue
Fullsend->>Jira: post_script report comment
Flow diagram for CI-result-is-data verify-pr gatingflowchart TD
A[Pull request event] --> B[Wait for other checks]
B --> C{Checks terminal?}
C -->|success, failure, or other terminal result| D[Dispatch verify-pr]
C -->|no checks| D
C -->|45-minute timeout| E[Skip dispatch]
D --> F{Jira issue in Review with ai-generated-jira label?}
F -->|yes| G[Run sandboxed verify-pr]
F -->|no| H[Skip agent]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
🤖 Finished Verify Pr · ✅ Success · Started 10:07 AM UTC · Completed 10:15 AM UTC Commit: Runtime: claude · Model: claude-opus-4-8 · Effort: high · Cost: $4.15 |
Verification Report for TC-6193 (commit 3c951de)
Overall: WARNThis documentation-only PR expands fullsend.md to cover CI install (github setup, inference provision, org GitHub App), the inline-wait dispatch design (CEL trigger + pre_script Jira gating), and the version-bump / plugin-version-sync upgrade procedures — satisfying all four acceptance criteria. Scope, diff size, commit traceability, and the secret scan all pass. The Overall: WARN is driven solely by CI Status: the only non-passing checks are the verify-pr harness's own self-referential runs (a prior attempt reported failure and the current run is in_progress); every substantive validation check passes. No code change requests, CI-failure defects, or eval regressions were found, so no sub-tasks were created. Merge remains a human decision. This comment was AI-generated by sdlc-workflow/verify-pr v0.13.9. |
Summary
Documents the deployed verify-pr fullsend install and day-2 operations in
fullsend.md(TC-6180 Req 16). Docs-only change.fullsend github setup,fullsend inference provision(hosted Vertex via WIF), and org GitHub App install.pull_request+lewagon/wait-on-check-action(inline CI wait) instead of check_suite/check_run/workflow_run; CI-result-is-data-not-a-gate; the CEL trigger on the composing child; pre_script Jira JQL gating.vars.*/secrets.*(incl. the Jira Service AccountJIRA_EMAIL/JIRA_API_TOKEN) with how/when to rotate each.fullsend_version: "v0.43.0"); bump + re-validate procedure.plugin.json+marketplace.jsonbumped together.v0.37.0framing in the intro/prerequisites.Head branch is on the upstream repo (not the fork) so the verify-pr
pull_requestdispatch gets secrets + OIDC and can run. Supersedes #304.Verification
Reviewed for accuracy against the deployed sources:
.github/workflows/fullsend-verify-pr.yml,.fullsend/config.yaml,.fullsend/config.base.yaml,.fullsend/harness/verify-pr.yaml, and the two version files. CI checks (CONVENTIONS.md) all green:pytest147 passed,claude plugin validate✔,skillsaw0 errors.Implements TC-6193
🤖 Generated with Claude Code
Summary by Sourcery
Document the fullsend verify-pr CI deployment, dispatch design, credential rotation, and upgrade procedures.
Enhancements:
Documentation: