Skip to content

chore(#62): onboard react-shopping-cart - #63

Merged
ralphbean merged 1 commit into
mainfrom
agent/62-onboard-react-shopping-cart
Nov 13, 2026
Merged

ralphbean merged 1 commit into
mainfrom
agent/62-onboard-react-shopping-cart

Conversation

@fullsend-ai-coder

Copy link
Copy Markdown
Contributor

Summary

Brings the transferred demo repo fullsend-ai/react-shopping-cart under org governance.

  • peribolos.yaml: adds a react-shopping-cart repo block with shairevivo as a direct admin collaborator, and grants the repo to the usual teams: admins (admin), maintainers (maintain), triagers (triage), writers (write).
  • inference-gateway/allowlist.yaml + re-rendered config.yaml: enrolls the repo, which already carries the fullsend workflows. Its ids (1443063811 / owner 271249281) come from gh api repos/fullsend-ai/react-shopping-cart.
  • There's no repos/react-shopping-cart.yml, because the org ruleset baseline is enough.

Please confirm

  • "Owner" isn't a repo-level role, so shairevivo gets admin, the highest repo permission.
  • Peribolos is total-state. Any collaborator carried over in the transfer other than shairevivo will be removed on apply. The sandbox couldn't list the repo's current collaborators (403).
  • The gateway allowlist entry lets this repo's workflows call the paid OpenAI-backed gateway. Drop that part if you don't want it yet.
  • Open PR feat(#1): add fullsend-triagers and fullsend-writers teams #30 also edits the teams section of peribolos.yaml, so whichever PR lands second may need a rebase.

Testing

  • python3 scripts/lint.py: passed.
  • python3 -m unittest discover -s inference-gateway: 11 tests passed.
  • python3 inference-gateway/render.py: the rendered config.yaml is committed.
  • scan-secrets: passed.
  • I used Python 3.14 because 3.15 isn't available in the sandbox.
  • Not run:
    • pre-commit: it couldn't fetch its hook repos (TLS error). Ruff and shuck don't apply to YAML.
    • render.py --verify-ids: it stopped at the existing epic-creator-eval entry, which this token can't read. The new entry's ids were checked directly.
    • The peribolos dry run needs an org token, so a maintainer needs to run it. It should plan only the five additions above, with no removals.

🤖 Generated with Claude Code


Closes #62

Post-script verification

  • Branch is not main/master (agent/62-onboard-react-shopping-cart)
  • Secret scan passed (gitleaks — ce2fd1c7567088c30f8600f9ff83b89caee52a39..HEAD)
  • PR body secret scan passed (gitleaks — no-git)
  • Final PR description secret scan passed (gitleaks — no-git)

The demo repo fullsend-ai/react-shopping-cart was transferred into
the org but is not in the governance config.

peribolos.yaml: add a repo block with shairevivo as a direct admin
collaborator ("owner" is not a repo role; admin is the highest), and
grant the repo to the admins (admin), maintainers (maintain),
triagers (triage) and writers (write) teams.

inference-gateway: the repo carries the fullsend workflows, so add it
to allowlist.yaml (id 1443063811, owner 271249281, from the GitHub
API) and re-render config.yaml.

No repos/ ruleset file: the org baseline is enough.

Peribolos is total-state: any collaborator carried over in the
transfer other than shairevivo is removed on apply. The current
collaborator list could not be read from the sandbox (403).

pre-commit could not fetch remote hook repos (TLS error). Ran
scan-secrets, scripts/lint.py and the render tests directly
(Python 3.14; 3.15 not available); all passed. ruff/shuck do not
apply to YAML. render.py --verify-ids could not read the existing
epic-creator-eval entry (no access). The peribolos dry run was not
run (no org token); it should plan only the additions above.

Closes #62

Assisted-By: Claude Opus 5.5(Claude Code)
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@fullsend-ai-coder
fullsend-ai-coder Bot requested a review from a team as a code owner November 13, 2026 16:26
@fullsend-ai-coder fullsend-ai-coder Bot added the ready-for-review Triggers review agent dispatch label Nov 13, 2026
@fullsend-ai-review

fullsend-ai-review Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

🤖 Finished Review · ✅ Success · Started 4:28 PM UTC · Completed 4:33 PM UTC

Commit: 8596f9d · View workflow run →

Runtime: pi · Model: sonnet → claude-sonnet-5-5 · Effort: high · Cost: $0.80

@fullsend-ai-review fullsend-ai-review Bot added the risk/low PR risk: low label Nov 13, 2026
@fullsend-ai-review

Copy link
Copy Markdown

Risk Assessment: low (1/5)

Details

Small additive configuration change by an automated bot with no protected path changes, low blast radius, and full alignment with the linked issue.

@fullsend-ai-review

Copy link
Copy Markdown

Looks good to me

@fullsend-ai-review fullsend-ai-review Bot added the ready-for-merge All reviewers approved — ready to merge label Nov 13, 2026
@ralphbean
ralphbean merged commit 1646b8a into main Nov 13, 2026
65 of 66 checks passed
@ralphbean
ralphbean deleted the agent/62-onboard-react-shopping-cart branch November 13, 2026 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-merge All reviewers approved — ready to merge ready-for-review Triggers review agent dispatch risk/low PR risk: low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Onboarding a new repo, already transferred

1 participant