mirror of
https://github.com/vitorpamplona/amethyst.git
synced 2026-10-05 19:28:25 +00:00
Claude
2b2e47256b
Completes the store-implementer skill set requested by the vespa-eventstore consumer: the NIP-85 trust-assertion model (kind map, tag vocabulary, value semantics, authorization conventions) and the NIP-50 indexing surface (the SearchableEvent contract plus an exhaustive kind -> indexableContent table external search engines can diff at version bumps). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01ADBfjWhsXyHZb7ea2eeBhJ
115 lines
7.1 KiB
Markdown
115 lines
7.1 KiB
Markdown
# Amethyst Skill Library — History & Changelog
|
|
|
|
> Historical record of how the `.claude/skills/` library was built and audited.
|
|
> The 8 original skills were created in 2025 using the skill-creator 10-step
|
|
> methodology (detailed per-skill progress logs pruned in 2026-06 — see git
|
|
> history of this file if you need them). For the current skill list and how
|
|
> the two skill layers (codebase-oriented vs technique-oriented) fit together,
|
|
> see the Skills section of `.claude/CLAUDE.md`.
|
|
|
|
## Phase 1 (2025): Core skills created
|
|
|
|
Eight hybrid domain skills (general expertise + Amethyst-specific patterns),
|
|
each with a SKILL.md plus bundled `references/` and `scripts/`:
|
|
|
|
1. **kotlin-multiplatform** — KMP architecture, the jvmAndroid source-set pattern, expect/actual catalog
|
|
2. **gradle-expert** — build system, version catalog, dependency troubleshooting
|
|
3. **kotlin-expert** — Flow state, sealed hierarchies, @Immutable, DSL builders
|
|
4. **compose-expert** — shared composables, state management, Material3, ImageVector
|
|
5. **desktop-expert** — Desktop UX, window management, Compose Desktop APIs
|
|
6. **android-expert** — Android navigation, permissions, platform APIs
|
|
7. **nostr-expert** — Nostr protocol, Quartz implementation, NIPs, events, tags
|
|
8. **ios-expert** — ⏸️ deferred (iOS targets are mature, but no iOS-specific UI work has surfaced in this repo yet)
|
|
|
|
## Phase 2 (2026-04): Audit & Expansion
|
|
|
|
After a full audit of the skill library, the following changes were made:
|
|
|
|
### Stale references fixed
|
|
- `CLAUDE.md` tech-stack versions replaced with a pointer to `gradle/libs.versions.toml` as the source of truth.
|
|
- `kotlin-multiplatform` reframed iOS as a mature target (not future) and added secp256k1-kmp version notes.
|
|
- `desktop-expert` Main.kt line references rewritten to match current layout (NavigationRail moved to `ui/deck/SinglePaneLayout.kt`); the obsolete "hardcoded ctrl = true anti-pattern" section replaced with a note that `isMacOS` branching is now applied throughout.
|
|
|
|
### Redundant files removed
|
|
- `.claude/skills/compose-desktop.md` deleted (superseded by `desktop-expert/`).
|
|
|
|
### New references added to existing skills
|
|
- `nostr-expert/references/nip19-bech32.md` — `Nip19Parser`, `Bech32Util`, `TlvBuilder`, entities.
|
|
- `nostr-expert/references/event-factory.md` — `EventFactory` dispatch + registering a new kind.
|
|
- `nostr-expert/references/crypto-and-encryption.md` — `EventHasher`, `Secp256k1Instance`, NIP-44, `SharedKeyCache`.
|
|
- `nostr-expert/references/large-cache.md` — `LargeCache<K,V>` + `ICacheOperations`.
|
|
- `kotlin-expert/references/common-utilities.md` — `NumberFormatters`, `TimeUtils`, `Hex`, `PubKeyFormatter`, `CoroutinesExt.launchIO`, etc.
|
|
- `compose-expert/references/rich-text-parsing.md` — `RichTextParser`, `UrlParser`, `GalleryParser`, NIP-92 imeta.
|
|
- `android-expert/references/image-loading.md` — Coil 3.x setup, custom fetchers, `MyAsyncImage`, `RobohashAsyncImage`.
|
|
|
|
### New skills created
|
|
- **`account-state/`** — `Account.kt` (50+ StateFlow properties) and `LocalCache.kt` event store.
|
|
- **`relay-client/`** — `ComposeSubscriptionManager`, filter assemblers, preloaders.
|
|
- **`feed-patterns/`** — `FeedFilter`, `AdditiveComplexFeedFilter`, `FeedViewModel` hierarchy in `commons/`.
|
|
- **`auth-signers/`** — `NostrSigner` abstraction across internal, NIP-46 remote, and NIP-55 external signers.
|
|
|
|
## Phase 3 (2026-06): Fable 5 config review
|
|
|
|
Instructions written to coach older models were removed now that the model
|
|
handles them natively; stale references fixed:
|
|
|
|
- `CLAUDE.md`: deleted the 5-step skill-approval "Workflow" section
|
|
(skills auto-trigger; the approval loop blocked autonomous sessions);
|
|
condensed "Verify, Don't Guess" to the repo-specific tooling pointers and
|
|
dropped references to `/bugfix` / `/investigate` (never committed to this
|
|
repo); replaced the mandated emoji survey matrix with one-line guidance.
|
|
- `android-expert` and `desktop-expert` SKILL.md gained YAML frontmatter —
|
|
without it they were listed without trigger descriptions and never
|
|
auto-invoked.
|
|
- `commands/extract.md`: fixed stale `shared-ui/` module name → `commons/`.
|
|
- `skills/quartz-kmp.md` breadcrumb deleted (KMP migration long complete;
|
|
`quartz-integration` and `nostr-expert` cover its pointers).
|
|
- Stop hook moved to `.claude/hooks/stop-spotless.sh` and gated on modified
|
|
Kotlin files, so Q&A-only turns no longer pay a Gradle invocation.
|
|
|
|
Second audit pass (every concrete claim checked against the code; `amy-expert`,
|
|
`find-missing-translations`, `find-non-lambda-logs`, and the vendored technique
|
|
skills verified clean):
|
|
|
|
- `auth-signers`: bunker login entry point corrected — `NostrSignerRemote.fromBunkerUri(...)`
|
|
+ `connect()`, not the nonexistent `RemoteSignerManager.connect(url)`.
|
|
- `nostr-expert`: NIP count 57 → 80+ packages; `Nip44v2.encrypt/decrypt`
|
|
static-object snippet replaced with the real `Nip44` facade
|
|
(returns `EncryptedInfo`, `encodePayload()` for event content); invented
|
|
`Nip19.npubEncode`/`Nip19Result` API replaced with the real `ByteArray`
|
|
extensions (`toNpub()`, …), entity `create()` helpers, and
|
|
`Nip19Parser.uriToRoute()?.entity`.
|
|
- `nostr-expert/references/nip-catalog.md`: heading count (60+8) replaced with
|
|
actual package counts (87 + 23 experimental) and a ground-truth pointer.
|
|
- `quartz-integration`: NIP-19 decode example rewritten for
|
|
`ParseReturn.entity` (the `Nip19Parser.Return.*` sealed class never existed);
|
|
Event Store section corrected from "Android only" to commonMain/all platforms
|
|
with the real `store.sqlite.EventStore` import and suspend generic `query<T>`.
|
|
|
|
## Phase 4 (2026-08): Store-implementer skills (external consumer request)
|
|
|
|
Three skills added at the request of an external Quartz consumer
|
|
(vespa-eventstore — a server-side `IEventStore` on Vespa that asserts result
|
|
parity against the SQLite store in CI). All three document the
|
|
**store/relay-implementer's perspective**, which `quartz-integration` and
|
|
`nostr-expert` (client-side) did not cover. Requirements doc: the skill-requests
|
|
file reviewed 2026-08-04; the requester's items #4 (storage-lifecycle-nips) was
|
|
folded into `event-store-semantics` per their own recommendation, and #5
|
|
(relay-server/geode policies) was declined as not currently needed.
|
|
|
|
- **`event-store-semantics/`** — the `IEventStore`/SQLite-store behavioral
|
|
contract as named rules (STORE-Fxx/Wxx/Dxx/Cxx/Sxx/Nxx) with a semantics
|
|
changelog for pin-bump review. Written from `QueryBuilder`,
|
|
`MergeQueryExecutor`, the seven `*Module.kt` files, and `IEventStore` KDoc.
|
|
- **`nip85-trusted-assertions/`** — the NIP-85 model (10040/30382/30383/30384/
|
|
30385), full tag vocabulary with value semantics, authorization conventions,
|
|
worked JSON examples, stability notes.
|
|
- **`searchable-events/`** — the `SearchableEvent` contract + maintenance
|
|
mandate, with `references/searchable-kinds.md` holding the exhaustive
|
|
kind → class → `indexableContent()` table (126 classes / 129 kinds) that
|
|
external search engines diff at version bumps.
|
|
|
|
Follow-ups suggested but not implemented: a shared JSON test-vector corpus for
|
|
filter semantics (testFixtures both the SQLite tests and external parity suites
|
|
could run), and a snapshot test pinning the searchable-kind set.
|