Make the skill trigger reliably: broader description, dependency-detection hook, doc fixes - #111
Merged
Merged
Conversation
The skill matcher keys off signals present in the conversation. When a requirement lives inside a fetched issue/PR/spec body (e.g. "Implement #85.") rather than the user's own words, none of the old trigger keywords appear in the prompt and the skill never fires. Widen the description to trigger on indirect requests, on in-context code signals (SpringPlaywrightIT, @PlaywrightElement, IT file paths), and any Vaadin-project task involving a Playwright/IT test. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…steps - element-mapping.md: verify every method in the Key Methods column against the auto-generated api-reference.md; replace non-existent APIs (selectByText, clickRow, sortByColumn, assertUnchecked, sendMessage, uploadFile, wrong get(page) factories) with the real ones - SKILL.md: fix wrong wrapper calls in the raw-locator guidance, restore the formatter-mangled Java templates, document MainView/Main root route mapping, add a version-drift caveat for api-reference.md, and make Steps 4-5 write/run without confirmation during autonomous execution - setup.md: clean up malformed XML snippets - TESTING.md: describe page objects by library package instead of a source path that only exists in the DramaFinder repo Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Skill matching is a heuristic over context: a task that never mentions Vaadin/Playwright/tests gives the model no signal to load the skill. Add two deterministic signal sources: - SessionStart plugin hook that greps pom.xml for the dramafinder artifact and, on a hit, injects a standing reminder to use the skill for any IT test work - setup.md step that appends an equivalent pointer to the consuming project's CLAUDE.md, covering sessions where the plugin hook isn't active Bump plugin version to 0.3.0 (plugin.json + tile.json). Co-Authored-By: Claude Fable 5 <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
Three related changes to make the
vaadin-playwright-testskill fire and behave correctly:Broaden the skill trigger description so it also matches indirect requests — GitHub issues, PRs, specs, "fix the failing test" — not just direct mentions of DramaFinder/Playwright.
Add deterministic trigger signals for tasks that never mention Vaadin/tests at all. Description matching is a heuristic over context; if nothing in the prompt or read files hints at testing, no wording helps. So:
SessionStartplugin hook grepspom.xmlfor thedramafinderartifact and, on a hit, injects a standing reminder to use the skill for any IT test work;CLAUDE.md, covering sessions where the plugin hook isn't active.Fix documentation inaccuracies found in review:
element-mapping.md: the Key Methods column listed many methods that don't exist (selectByText,clickRow,sortByColumn,assertUnchecked,sendMessage,uploadFile, wrongget(page)factories on Dialog/Details/Card/MenuBar/…). Every entry is now verified against the auto-generatedapi-reference.md.SKILL.md: fixed wrong wrapper calls in the raw-locator guidance, restored the formatter-mangled Java example blocks, documented theMainView/Main→ root route mapping, added a version-drift caveat for the bundled API reference, and made Steps 4–5 write/run without confirmation during autonomous execution so the broadened trigger doesn't stall unattended agents.setup.md: cleaned up malformed XML snippets.TESTING.md: page objects described by library package instead of a source path that only exists inside this repo.Test plan
🤖 Generated with Claude Code