Improve analyze-and-plan skill quality (65% → 94%)#56
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Improves the “analyze-and-plan” skill documentation to drive more structured requirements analysis and acceptance criteria for AEM Edge Delivery Services tasks.
Changes:
- Expanded YAML frontmatter description and clarified “use when…” triggers for the skill.
- Reworked the workflow to include a structured acceptance-criteria template and a validation checkpoint.
- Extracted task-type analysis guidance into a new
resources/task-types.mdfile.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
skills/aem/edge-delivery-services/skills/analyze-and-plan/SKILL.md |
Streamlines steps, adds a standardized acceptance criteria format + validation checklist, and references external resource docs. |
skills/aem/edge-delivery-services/skills/analyze-and-plan/resources/task-types.md |
New consolidated task-type guidance referenced by the main skill doc. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
6bcce2d to
dc553f9
Compare
dc553f9 to
d90b0af
Compare
d57aeb0 to
4433b97
Compare
81087dc to
2a7e37d
Compare
shsteimer
approved these changes
Apr 10, 2026
2873885 to
2c289e3
Compare
Agent-Id: agent-cf9473ea-8773-40a5-a76b-4d4a3c1d69ea
Agent-Id: agent-4edc780f-88d7-41bb-9165-da69b54f9fe0 Linked-Note-Id: 4c1224e8-045f-4aa9-a90d-d304af6da437
Agent-Id: agent-47fa28be-5997-4350-aa63-93236a73a9a8 Linked-Note-Id: b26d1d82-8ddb-4918-be52-3d6b78f18af8
Agent-Id: agent-c1d2bcc9-d1ea-4b25-be60-9af5369cba63 Linked-Note-Id: 99d5b8bf-54aa-4131-8014-4aba9d273c6d
Signed-off-by: Lars Trieloff <lars@trieloff.net>
Signed-off-by: Lars Trieloff <lars@trieloff.net>
Agent-Id: agent-799ea216-b518-49a7-871d-f851b2c8f1e4 Linked-Note-Id: 3e74f377-1037-43ba-9ab3-90b2d13864f1
Signed-off-by: Lars Trieloff <lars@trieloff.net>
Agent-Id: agent-3200527d-e95f-40b7-b6b4-0867f55897cc Linked-Note-Id: 7ee09537-c8e4-4de5-ae2e-d8d48fb80821
Signed-off-by: Lars Trieloff <lars@trieloff.net>
2c289e3 to
26009d0
Compare
Tessl Eval Results
Triggered by commit eval: re-run after removing --workspace flag |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Improves the weakest-scoring skill in the repo using Tessl's
--optimizeworkflow.Changes
skills/aem/edge-delivery-services/skills/analyze-and-plan/SKILL.mdskills/aem/edge-delivery-services/skills/analyze-and-plan/resources/task-types.md(new)Tessl Review Score