Skip to content

#215 page-decomposition: add page-import pipeline guard + Use-this-when trigger - #244

Merged
trieloff merged 1 commit into
mainfrom
trieloff/skill-desc-page-decomposition
Jul 6, 2026
Merged

#215 page-decomposition: add page-import pipeline guard + Use-this-when trigger#244
trieloff merged 1 commit into
mainfrom
trieloff/skill-desc-page-decomposition

Conversation

@trieloff

@trieloff trieloff commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Addresses #215.

Problempage-decomposition said "Invoked per section" but had no guard against direct invocation and led with what it does rather than when to invoke it.

Solution — Rewrote the description to lead with a Use this when … trigger and added an explicit Do not invoke directly — called by page-import per section 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 Skill Lint

⚠️ edge-delivery-services — 1 warning(s)
⚠ tile.json is deprecated. Migrate to .tessl-plugin/plugin.json to publish a plugin-shape package. Run `tessl plugin migrate` to convert automatically. tile.json support will be removed in a future release.

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

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

Updated by tessl-lint for commit 3299ca1.

@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:06
@trieloff
trieloff requested a review from shsteimer as a code owner July 6, 2026 18:06
Copilot AI review requested due to automatic review settings July 6, 2026 18:06

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 page-decomposition skill’s frontmatter description to improve routing reliability and prevent accidental direct invocation, aligning with the guidance in #215 for pipeline sub-skills.

Changes:

  • Rewrites the opening description to start with a clear “Use this when …” trigger.
  • Adds an explicit “Do not invoke directly — called by page-import per section” guard to clarify pipeline-only usage.

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

@trieloff
trieloff merged commit 2a20755 into main Jul 6, 2026
10 checks passed
@trieloff
trieloff deleted the trieloff/skill-desc-page-decomposition branch July 6, 2026 18:15
@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