Skip to content

feat: add fullsend-architecture skill#1710

Draft
waynesun09 wants to merge 1 commit into
mainfrom
feat-skill-fullsend-architecture
Draft

feat: add fullsend-architecture skill#1710
waynesun09 wants to merge 1 commit into
mainfrom
feat-skill-fullsend-architecture

Conversation

@waynesun09
Copy link
Copy Markdown
Contributor

Summary

  • Adds skills/fullsend-architecture/SKILL.md covering fullsend domain knowledge that agents frequently get wrong
  • Covers per-org vs per-repo install modes, mint vs inference WIF pool separation, and GCP resource boundaries
  • Includes 10 common-mistake entries with WRONG/RIGHT examples and key code location references

Test plan

  • Verify skill loads correctly when referenced by agents
  • Spot-check WRONG/RIGHT examples against current codebase
  • Confirm no internal Red Hat content leaked into the skill

…separation

Add a skill covering fullsend domain knowledge that agents frequently
get wrong: per-org vs per-repo install modes, mint vs inference WIF pool
separation, and GCP resource boundaries. Includes 10 common-mistake
entries with WRONG/RIGHT examples and key code location references.

Signed-off-by: Wayne Sun <gsun@redhat.com>
@github-actions
Copy link
Copy Markdown

Site preview

Preview: https://a02dfdf7-site.fullsend-ai.workers.dev

Commit: a7fc63a01b2d726c105aca601b6413793c0994e9

@fullsend-ai-review
Copy link
Copy Markdown

Review

Findings

No findings.

All factual claims in the skill were verified against the current codebase — install mode branching logic, default role lists, WIF pool/provider names, flag scoping, Cloud Function env vars, and key code locations are all accurate. No sensitive data or internal references detected. Skill structure and frontmatter follow established patterns (consistent with mint-enroll, agent-scaffolding, etc.).

@fullsend-ai-review fullsend-ai-review Bot added the ready-for-merge All reviewers approved — ready to merge label May 30, 2026
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant