fix(aem-eds): clean up tile to clear all lint warnings#104
Open
fix(aem-eds): clean up tile to clear all lint warnings#104
Conversation
Two skills existed in the skills/ directory but were absent from tile.json, so tessl never saw them. Adds both to the skills map and creates the missing package.json for ue-component-model so the release workflow can pick it up. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Suppresses tessl lint warnings about CHANGELOG.md and package.json files that exist as semantic-release / semantic-release-monorepo artifacts but are not skill content. Negation pattern preserves package.json files under scripts/ so that future skills shipping node script tools have their packaging metadata bundled correctly. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2 tasks
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Renames resources/ to references/ across 8 skills to match the agentskills.io spec — every other plugin in this repo and EDS's own ue-component-model already use references/. Converts backtick-quoted resource paths in SKILL.md to markdown links so tessl lint sees them as reachable from manifest entrypoints, eliminating all 17 orphan warnings. Also fixes two pre-existing broken cross-skill references that pointed at ../page-import/resources/... (no such directory); the actual file lives in content-driven-development/references/. Adds a markdown link in code-review/SKILL.md to its previously unreferenced references/review-checklist.md. Per Anthropic's agent skills spec, markdown links don't trigger eager loading — Claude reads SKILL.md, sees the link, decides on demand whether to follow it. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This was referenced Apr 28, 2026
shsteimer
added a commit
that referenced
this pull request
Apr 28, 2026
Adds a trailing comment marker to page-import SKILL.md so the tessl-lint workflow (introduced in #106) sees a tile change and runs against the EDS tile in its current main-branch state (with all the warnings PR #104 will fix). Demonstrates how warnings surface in the GitHub UI. This PR will be closed without merging. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.
Summary
Five fixes that take the EDS plugin tile to a clean lint state (only the unrelated `code-review` token-bloat warning remains, deferred to a follow-up):
Per Anthropic's agent skills spec, markdown links don't trigger eager loading — Claude reads SKILL.md, sees the link, decides on demand whether to follow it. Confirmed in lint output: on-demand token range now shows lazy bounds (e.g., building-blocks `2.3k-6.7k`) rather than a single value, with the lower bound unchanged from before.
Verification
Before:
```
⚠ Found 36 orphaned file(s) not linked from manifest entrypoints
⚠ Found 20 skill file(s) outside spec directories
⚠ Skill 'code-review': SKILL.md is approximately 10277 tokens
```
After:
```
⚠ Skill 'code-review': SKILL.md is approximately 10315 tokens
✔ Tile adobe/aem-edge-delivery-services@0.1.0 is valid
```
`tessl skill publish plugins/aem/edge-delivery-services --dry-run` confirms a valid tile with all 20 skills declared.
Follow-ups (separate PRs)
🤖 Generated with Claude Code