feat(lavish): add lavish-axi HTML artifact review skill plugin#227
Conversation
Installs the lavish skill via skills.sh from github.com/kunchenguid/lavish-axi (MIT license). Wires marketplace entry, release-please tracking, and README. - plugins/lavish/: skill plugin dir with SKILL.md, skills-lock.json, and multi-runtime manifests (.claude-plugin, .codex-plugin, root plugin.json) - .claude-plugin/marketplace.json: source-of-truth marketplace entry added - .agents/plugins/marketplace.json: Codex marketplace entry added - release-please-config.json + .release-please-manifest.json: lavish package - README.md: Built-in Plugins entry Diff is scoped to lavish only; unrelated multi-format drift was reverted.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughA new ChangesLavish Plugin Addition
Sequence DiagramsequenceDiagram
participant Agent
participant lavish-axi
participant Human
Agent->>lavish-axi: npx lavish-axi open (create HTML artifact)
lavish-axi-->>Agent: session opened
loop Poll for feedback
Agent->>lavish-axi: npx lavish-axi poll
lavish-axi-->>Agent: annotations / layout_warnings
Agent->>Agent: apply feedback or fix layout_warnings
Agent->>lavish-axi: npx lavish-axi --agent-reply
end
Human->>lavish-axi: approve artifact
Agent->>lavish-axi: npx lavish-axi end
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Up to standards ✅🟢 Issues
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request registers and configures a new plugin called 'lavish' across the marketplace manifests, release-please configuration, and the main README.md, alongside adding its skill files and lockfile. The reviewer pointed out that the 'lavish' plugin installation command is missing from the 'Built-in plugins' list at the bottom of README.md and suggested adding it to maintain consistency.
# Conflicts: # .claude-plugin/marketplace.json
🔍 Tessl Skill Review
|
| Dimension | Score | Detail |
|---|---|---|
| conciseness | ██░ 2/3 | There is significant repetition throughout the skill: the 'when to use' criteria appear in both the 'When to use' section and the 'Commands & rules' section; the poll behavior and npx invocation pattern are explained multiple times; the playbook guidance is stated twice nearly verbatim. However, the content is mostly domain-specific and doesn't over-explain concepts Claude already knows. |
| actionability | ███ 3/3 | The skill provides concrete, copy-paste-ready commands (npx -y lavish-axi , poll, end, stop, playbook ), specific file paths (.lavish/.html), and clear behavioral instructions. The workflow steps are executable and the design system priority order is specific and actionable. |
| workflow clarity | ███ 3/3 | The workflow is clearly sequenced (create HTML → open session → poll → fix layout warnings → apply feedback → poll again → end), includes explicit validation checkpoints (fix layout_warnings before involving the human), and has error recovery guidance (re-run poll if killed, queued feedback is never lost). The feedback loop is well-defined. |
| progressive disclosure | ██░ 2/3 | The skill effectively references playbooks via npx -y lavish-axi playbook <id> for detailed guidance, which is good progressive disclosure. However, the main SKILL.md itself has redundant sections (Commands & rules largely repeats Workflow and other sections), and without bundle files, the playbook references can't be verified. The content that is inline could be better organized to reduce duplication. |
Overall: This is a well-structured skill with strong actionability and clear workflow sequencing including validation checkpoints and feedback loops. Its main weakness is redundancy—several instructions (poll behavior, npx invocation, when-to-use criteria, playbook guidance) are stated multiple times across different sections, inflating token cost without adding clarity. The progressive disclosure via playbook commands is a smart pattern, though the inline content itself could be tightened.
Suggestions:
- Consolidate duplicated content: the poll behavior, npx -y prefix rule, when-to-use criteria, and playbook guidance each appear in multiple sections. Keep each instruction in one authoritative location only.
- Remove the 'Commands & rules' section or restructure it as a quick-reference table, since most of its content duplicates the Workflow and other sections above it.
To improve your score, point your agent at the Tessl optimization guide. Need help? Jump on our Discord.
Feedback
Report issues with this review at tesslio/skill-review, or send private feedback from your terminal with tessl feedback.
Address gemini-code-assist review: the Built-in plugins install block listed dev3000 but not lavish.
|
✅ @amondnet님의 요청을 완료했습니다 (1m 39s) —— View job PR Review
SummaryThe README fix is confirmed —
Overall PR Assessment ✅The PR is well-structured and follows repository conventions correctly:
One NoteThe Tessl skill reviewer gave LGTM — the plugin is wired up correctly across all four runtimes and the README is now complete. |
|
1 participant
Summary
Adds the
lavishbuilt-in skill plugin to the marketplace. The skill is sourcedfrom github.com/kunchenguid/lavish-axi
(MIT) and installed via skills.sh (
bunx skills add).Changes
plugins/lavish/— new plugin directory with:.agents/skills/lavish/SKILL.md— skills.sh-managed skill (vendor-synced, do not edit directly)skills-lock.json— skills.sh lock file.claude-plugin/plugin.json— Claude Code manifest.codex-plugin/plugin.json— Codex manifestplugin.json(root) — Antigravity manifest.claude-plugin/marketplace.json— source-of-truth marketplace entry added.agents/plugins/marketplace.json— Codex marketplace entry added (generated)release-please-config.json—plugins/lavishpackage entry with extra-files coverage.release-please-manifest.json—plugins/lavishversion initializedREADME.md— Built-in Plugins table entry addedNotes
bunx skills updateinsideplugins/lavish/bun scripts/cli.ts multi-formatwas reverted before committingSKILL.mdfile is vendor-synced; fix upstream at kunchenguid/lavish-axi rather than editing it directlySummary by cubic
Adds
lavish, an HTML artifact review skill powered bylavish-axi, so agents can turn complex outputs into rich, reviewable pages. Registers the plugin in marketplaces, hooks it into release automation, and updates docs (including the bulk install list).New Features
plugins/lavish/with vendor-synced.agents/skills/lavish/SKILL.md,skills-lock.json, and manifests for Claude/Codex/root.lavishin.claude-plugin/marketplace.jsonand.agents/plugins/marketplace.json.release-pleaseforplugins/lavishinrelease-please-config.jsonand.release-please-manifest.json./plugin install lavish@pleaseaito the built-in install block.Migration
/plugin install lavish@pleaseai.bunx skills updateinplugins/lavish/.Written for commit 6fb1e1a. Summary will update on new commits.
Summary by CodeRabbit
/plugin install lavish@pleaseai.