Skip to content

Commit 741efb6

Browse files
committed
fix: address code review — version bump, color labels, stale values
- Bump plugin version to 1.1.0 in plugin.json and CLAUDE.md - Fix course example referencing wrong font stack (Bricolage → Instrument Serif) - Fix color label mismatches in interactive-elements.md (vermillion → amber, forest → sky blue) to match Obsidian & Amber design-system.md - Replace stale hardcoded rgba(217,79,48) with var(--color-accent-glow) - Unify blue color value between changelog and design system (#60A0E0)
1 parent a8ccd9d commit 741efb6

5 files changed

Lines changed: 9 additions & 9 deletions

File tree

CLAUDE.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ These require Python 3.9+ and are only executed in GitHub Actions workflows.
8686
│ ├── build_search_index.py # Index builder
8787
│ ├── fetcher/ # Documentation fetching package (8 modules)
8888
│ └── lookup/ # Search and validation package (7 modules)
89-
├── plugin/ # Claude Code Plugin (v1.0.0)
89+
├── plugin/ # Claude Code Plugin (v1.1.0)
9090
│ ├── .claude-plugin/plugin.json # Plugin metadata
9191
│ ├── commands/docs.md # /docs slash command (lean router)
9292
│ ├── skills/

plugin/.claude-plugin/plugin.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "claude-docs",
3-
"version": "1.0.0",
3+
"version": "1.1.0",
44
"description": "Searchable local mirror of Claude documentation. Provides the /docs command for instant access to API references, guides, and tutorials. Auto-discovery Skill reads docs automatically when you ask Claude-related questions. Docs auto-update via SessionStart hook.",
55
"author": {
66
"name": "costiash"

plugin/skills/claude-docs-changelog/SKILL.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -116,8 +116,8 @@ Use these exact CSS variables from the course design system:
116116
--text-muted: #5C5852;
117117
--accent: #F0A050;
118118
--accent-dim: rgba(240, 160, 80, 0.08);
119-
--blue: #60B0E0;
120-
--blue-dim: rgba(96, 176, 224, 0.08);
119+
--blue: #60A0E0;
120+
--blue-dim: rgba(96, 160, 224, 0.08);
121121
--green: #50C8A0;
122122
--green-dim: rgba(80, 200, 160, 0.08);
123123
--purple: #C080E0;

plugin/skills/claude-docs-course/examples/course-from-docs.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,4 +55,4 @@ A single self-contained HTML file (~80-120KB) with:
5555
- Data flow animation of the hook lifecycle
5656
- Multiple-choice quizzes testing practical hook configuration
5757
- Glossary tooltips on terms: matchers, IIFE, ARIA, stdin, SSE, MCP
58-
- Warm design aesthetic (Bricolage Grotesque + DM Sans + JetBrains Mono)
58+
- Obsidian & Amber design aesthetic (Instrument Serif + Outfit + JetBrains Mono)

plugin/skills/claude-docs-course/references/interactive-elements.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -481,7 +481,7 @@ window.flowNext = function() {
481481
**CSS for active actor glow:**
482482
```css
483483
.flow-actor.active {
484-
box-shadow: 0 0 0 3px var(--color-accent), 0 0 20px rgba(217, 79, 48, 0.2);
484+
box-shadow: 0 0 0 3px var(--color-accent), 0 0 20px var(--color-accent-glow);
485485
transform: scale(1.05);
486486
transition: all var(--duration-normal) var(--ease-out);
487487
}
@@ -630,7 +630,7 @@ Same HTML/CSS/JS pattern as Multiple-Choice Quizzes, but with longer scenario de
630630
```
631631

632632
**Variants:**
633-
- `callout-accent`: vermillion left border, light accent background (for CS insights)
633+
- `callout-accent`: amber left border, light accent background (for CS insights)
634634
- `callout-info`: teal left border, light info background (for "good to know")
635635
- `callout-warning`: red left border, light error background (for common mistakes)
636636

@@ -1146,11 +1146,11 @@ A variant of the Group Chat Animation specifically designed for API and system i
11461146

11471147
| Role | Avatar | Color Variable | Use For |
11481148
|------|--------|---------------|---------|
1149-
| Client App | C | `--color-actor-1` (vermillion) | Your application code making API calls |
1149+
| Client App | C | `--color-actor-1` (amber) | Your application code making API calls |
11501150
| Claude API | A | `--color-actor-2` (teal) | The API gateway — auth, routing, rate limits |
11511151
| Claude (Model) | M | `--color-actor-3` (plum) | Claude's reasoning and responses |
11521152
| Tool Server | T | `--color-actor-4` (golden) | External tools, MCP servers, databases |
1153-
| User | U | `--color-actor-5` (forest) | The end user interacting with the app |
1153+
| User | U | `--color-actor-5` (sky blue) | The end user interacting with the app |
11541154

11551155
**Additional CSS for protocol variant:**
11561156
```css

0 commit comments

Comments
 (0)