Skip to content

#215 block-inventory: add page-import pipeline guard + Use-this-when trigger - #241

Merged
trieloff merged 2 commits into
mainfrom
trieloff/skill-desc-block-inventory
Jul 6, 2026
Merged

#215 block-inventory: add page-import pipeline guard + Use-this-when trigger#241
trieloff merged 2 commits into
mainfrom
trieloff/skill-desc-block-inventory

Conversation

@trieloff

@trieloff trieloff commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Addresses #215.

Problemblock-inventory's description opened with what it does, not when to invoke it, and gave no signal that it is a page-import pipeline step — so an agent could load it in isolation or skip it when routing.

Solution — Rewrote the description to lead with a Use this when … trigger and added an explicit Do not invoke directly — called by page-import guard, per the issue's recommendation for pipeline sub-skills. Content-only change to the frontmatter description.

Validated with npm run validate. Generated with AI assistance.


Pull Request opened by Augment Code | View session

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Tessl Plugin Lint

⚠️ edge-delivery-services — 1 warning(s)
⚠ Skill 'code-review': SKILL.md is approximately 10315 tokens (recommended maximum: 5000). Consider moving detailed content to separate reference files.

✔ Plugin adobe/aem-edge-delivery-services@0.1.0 is valid

✅ All 1 plugin(s) lint passed with 1 warning(s) total.

Updated by tessl-lint for commit 344e40a.

@trieloff trieloff added the ai-generated Pull request contains code generated or co-authored by AI tools label Jul 6, 2026
@trieloff
trieloff marked this pull request as ready for review July 6, 2026 18:07
@trieloff
trieloff requested a review from shsteimer as a code owner July 6, 2026 18:07
Copilot AI review requested due to automatic review settings July 6, 2026 18:07

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the block-inventory skill’s frontmatter description to improve routing and prevent accidental standalone invocation, aligning with the guidance in issue #215 for page-import pipeline sub-skills.

Changes:

  • Rewrites the description to start with a “Use this when …” trigger.
  • Adds an explicit “Do not invoke directly …” pipeline guard to indicate it’s a page-import pipeline sub-skill.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

---
name: block-inventory
description: Survey available blocks from local AEM Edge Delivery Services project and Block Collection to understand the block palette available for authoring. Returns block inventory with purposes to inform content modeling decisions.
description: "Use this when the page-import pipeline needs to survey the block palette available from a local AEM Edge Delivery Services project and the Block Collection to inform content modeling decisions. Covers returning a block inventory annotated with each block's purpose. Do not invoke directly — called by page-import as a pipeline step."
@trieloff
trieloff merged commit 3871f39 into main Jul 6, 2026
9 checks passed
@trieloff
trieloff deleted the trieloff/skill-desc-block-inventory branch July 6, 2026 18:14
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.0.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.2.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.0.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.0.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

ai-generated Pull request contains code generated or co-authored by AI tools released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants