Skip to content

docs(#4232): add AiResource typed variant package convention - #4233

Open
fullsend-ai-coder[bot] wants to merge 1 commit into
mainfrom
agent/4232-airesource-variant-convention
Open

docs(#4232): add AiResource typed variant package convention#4233
fullsend-ai-coder[bot] wants to merge 1 commit into
mainfrom
agent/4232-airesource-variant-convention

Conversation

@fullsend-ai-coder

Copy link
Copy Markdown
Contributor

Add an "AiResource typed variant packages" section to the ai-integrations workspace AGENTS.md documenting the discriminated-union package pattern: each spec.type variant (skill, rule, agent, mcp-server) gets its own dedicated catalog-model and catalog-backend-module package pair. This prevents future code agents from placing typed-variant types in ai-experience-common (as happened in PR #4164) and provides a 5-step checklist for creating new typed variants.


Closes #4232

Post-script verification

  • Branch is not main/master (agent/4232-airesource-variant-convention)
  • Secret scan passed (gitleaks — 51a6bc26febbc5a99dce25244441b8d1a5603a92..HEAD)
  • Pre-commit hooks passed (authoritative run on runner)
  • Tests ran inside sandbox

Add an "AiResource typed variant packages" section to the
ai-integrations workspace AGENTS.md documenting the
discriminated-union package pattern: each spec.type variant
(skill, rule, agent, mcp-server) gets its own dedicated
catalog-model and catalog-backend-module package pair. This
prevents future code agents from placing typed-variant types
in ai-experience-common (as happened in PR #4164) and provides
a 5-step checklist for creating new typed variants.

Closes #4232
@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Aug 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.40%. Comparing base (51a6bc2) to head (c0336bf).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4233   +/-   ##
=======================================
  Coverage   58.40%   58.40%           
=======================================
  Files        2438     2438           
  Lines       96960    96960           
  Branches    26937    26939    +2     
=======================================
  Hits        56628    56628           
  Misses      40135    40135           
  Partials      197      197           
Flag Coverage Δ *Carryforward flag
adoption-insights 84.55% <ø> (ø) Carriedforward from 51a6bc2
ai-integrations 69.76% <ø> (ø)
app-defaults 69.79% <ø> (ø) Carriedforward from 51a6bc2
augment 46.67% <ø> (ø) Carriedforward from 51a6bc2
boost 77.63% <ø> (ø) Carriedforward from 51a6bc2
bulk-import 72.79% <ø> (ø) Carriedforward from 51a6bc2
cost-management 13.55% <ø> (ø) Carriedforward from 51a6bc2
dcm 67.21% <ø> (ø) Carriedforward from 51a6bc2
extensions 56.59% <ø> (ø) Carriedforward from 51a6bc2
global-floating-action-button 71.18% <ø> (ø) Carriedforward from 51a6bc2
global-header 66.50% <ø> (ø) Carriedforward from 51a6bc2
homepage 47.50% <ø> (ø) Carriedforward from 51a6bc2
install-dynamic-plugins 59.95% <ø> (ø) Carriedforward from 51a6bc2
intelligent-assistant 75.24% <ø> (ø) Carriedforward from 51a6bc2
konflux 91.98% <ø> (ø) Carriedforward from 51a6bc2
lightspeed 69.02% <ø> (ø) Carriedforward from 51a6bc2
mcp-integrations 83.40% <ø> (ø) Carriedforward from 51a6bc2
orchestrator 66.91% <ø> (ø) Carriedforward from 51a6bc2
quickstart 63.74% <ø> (ø) Carriedforward from 51a6bc2
sandbox 79.56% <ø> (ø) Carriedforward from 51a6bc2
scorecard 86.23% <ø> (ø) Carriedforward from 51a6bc2
theme 88.77% <ø> (ø) Carriedforward from 51a6bc2
translations 5.12% <ø> (ø) Carriedforward from 51a6bc2
x2a 79.20% <ø> (ø) Carriedforward from 51a6bc2

*This pull request uses carry forward flags. Click here to find out more.


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 51a6bc2...c0336bf. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@gabemontero gabemontero left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@johnmcollier I'll defer to you on this one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-review Agent PR ready for human review workspace/ai-integrations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add AiResource discriminated-union package convention to ai-integrations AGENTS.md

1 participant