@agentskillkit/agent-skills 1.0.1 → 3.2.2

package/.agent/skills/git-conventions/examples/bad-commits.md

@@ -1,201 +0,0 @@
-# Bad Commit Message Examples
-
-## Anti-Pattern: Vague Messages
-
-### ❌ Too Short
-```
-fix: bug
-```
-**Problem**: What bug? Where? Why?
-
-**Should Be**:
-```
-fix(chart): correct Y-axis scaling for high volatility periods [doctrine: Law-4]
-
-Chart Y-axis was clipping during 50%+ price swings.
-Now dynamically adjusts with 10% padding.
-```
-
----
-
-### ❌ Generic Message
-```
-update: stuff
-```
-**Problem**: Meaningless. No type, no scope, no content.
-
-**Should Be**:
-```
-feat(api): add rate limiting to coin endpoints [doctrine: Performance]
-```
-
----
-
-### ❌ WIP Commits
-```
-wip
-```
-**Problem**: Never ship WIP to main branch. Squash before merge.
-
----
-
-## Anti-Pattern: Missing Critical Info
-
-### ❌ No Scope for Critical Changes
-```
-feat: add caching
-```
-**Problem**: Caching where? API? Frontend? What layer?
-
-**Should Be**:
-```
-feat(cache): implement Redis layer for coin metadata [doctrine: Performance]
-```
-
----
-
-### ❌ No Doctrine Tags
-```
-refactor: change chart data loading
-```
-**Problem**: Chart changes affect Laws 3 & 4. Tags required!
-
-**Should Be**:
-```
-refactor(chart): load data from historical API [doctrine: Law-3, Law-4]
-
-Separates realtime from historical data flow.
-```
-
----
-
-### ❌ Breaking Change Without Marking
-```
-feat(api): change price response format
-```
-**Problem**: This IS a breaking change but not marked!
-
-**Should Be**:
-```
-feat(api)!: change price response format [doctrine: Commercial]
-
-BREAKING CHANGE: Response now includes timestamp and source.
-
-Before: { price: 50000 }
-After: { currentPrice: 50000, timestamp: "...", source: "..." }
-```
-
----
-
-## Anti-Pattern: Wrong Format
-
-### ❌ Past Tense
-```
-feat(api): added caching layer
-```
-**Problem**: Should be imperative mood - "add", not "added"
-
-**Should Be**:
-```
-feat(api): add caching layer
-```
-
----
-
-### ❌ Capital First Letter
-```
-feat(api): Add caching layer
-```
-**Problem**: Description should be lowercase
-
-**Should Be**:
-```
-feat(api): add caching layer
-```
-
----
-
-### ❌ Period at End
-```
-feat(api): add caching layer.
-```
-**Problem**: No trailing punctuation
-
-**Should Be**:
-```
-feat(api): add caching layer
-```
-
----
-
-## Anti-Pattern: Multiple Changes
-
-### ❌ Kitchen Sink Commit
-```
-feat: add chart, fix login, update deps
-```
-**Problem**: Three unrelated changes. Split into 3 commits!
-
-**Should Be**:
-```
-# Commit 1
-feat(chart): add sparkline component [doctrine: Law-4]
-
-# Commit 2
-fix(auth): correct session timeout handling
-
-# Commit 3
-chore(deps): upgrade react to 18.2
-```
-
----
-
-## Anti-Pattern: Constitutional Violations
-
-### ❌ Describes Violation as Feature
-```
-feat(chart): inject realtime data to make charts more dynamic
-```
-**Problem**: This VIOLATES Law-3 and Law-4!
-
-**What Actually Happened**:
-- Developer didn't know the doctrine
-- PR reviewer should have caught this
-- Commit message reveals the violation
-
-**Correct Approach**:
-```
-feat(ui): add separate LivePriceTicker component [doctrine: Law-3]
-
-Shows realtime prices separately from historical charts.
-Charts remain honest representations of past data.
-```
-
----
-
-### ❌ Frontend Managing Truth
-```
-feat(state): add Redux store for portfolio value calculation
-```
-**Problem**: Violates Law-1! Backend owns financial truth.
-
-**Should Be**:
-```
-feat(api): add portfolio value endpoint [doctrine: Law-1]
-
-Backend calculates portfolio value with authoritative prices.
-Frontend only displays the result.
-```
-
----
-
-## Summary
-
-| Anti-Pattern | Why It's Bad | Fix |
-|--------------|--------------|-----|
-| Vague | No traceability | Be specific |
-| No scope | Can't identify affected area | Add scope |
-| No doctrine | Constitutional violation risk | Add tags |
-| Past tense | Violates Conventional Commits | Use imperative |
-| Multiple changes | Hard to revert, review | Split commits |
-| Describes violation | PR merger doesn't notice | Follow doctrines |

package/.agent/skills/git-conventions/examples/good-commits.md

@@ -1,135 +0,0 @@
-# Good Commit Message Examples
-
-## Feature Commits
-
-### API Feature
-```
-feat(api): add caching layer for coin price endpoints [doctrine: Performance]
-
-Implements Redis caching with 5-minute TTL for /api/coins/[id]/price
-to reduce external API load and improve response time.
-
-- P95 latency: 800ms → 50ms
-- External API calls reduced 80%
-
-Backend owns truth (Law-1) verified - cache is read-through only.
-```
-
-### Chart Feature
-```
-feat(chart): implement proper separation of historical and realtime data [doctrine: Law-3, Law-4]
-
-- Historical chart loads from /api/bitcoin/historical (immutable)
-- Realtime price shown in separate ticker component
-- Clear visual distinction for users
-
-Charts now represent truthful historical data without realtime noise.
-```
-
-### Mobile Feature
-```
-feat(mobile): implement swipe navigation for portfolio tabs [doctrine: Frontend-Mobile]
-
-Follows coinpika-swipe-tabs-doctrine:
-- Native scroll precedence maintained
-- No gesture conflicts
-- Predictable 60fps animations
-
-Ref: #234
-```
-
-## Fix Commits
-
-### Backend Fix
-```
-fix(api): prevent historical data mutation in price updates [doctrine: Law-2]
-
-Changed .push() to append-only insert to maintain immutability of
-historical price records.
-
-Before: priceHistory.push(newPrice) - mutates existing array
-After: INSERT INTO price_history - append only
-
-Fixes #456
-```
-
-### Chart Fix
-```
-fix(chart): remove realtime injection from BitcoinChart [doctrine: Law-3, Law-4]
-
-Charts must represent historical truth, not realtime excitement.
-Moved realtime price to separate LiveTicker component.
-
-Before: Chart received WebSocket ticks directly
-After: Chart loads from /api/historical, ticker shows live
-
-Fixes #789
-```
-
-### Mobile Fix
-```
-fix(mobile): restore native scroll on coin list [doctrine: Frontend-Mobile]
-
-Removed custom scroll implementation that blocked native gestures.
-Users can now scroll naturally without conflicts.
-
-Tested on: iOS 16, Android 13
-```
-
-## Refactor Commits
-
-### Data Refactor
-```
-refactor(data): separate realtime buffer from historical storage [doctrine: Law-3]
-
-Realtime WebSocket ticks now stored in Redis with 5-minute TTL.
-Backend aggregates ticks to OHLCV candles before persisting to Postgres.
-
-This enforces Law-3 (Realtime Ephemerality) and prevents contamination
-of historical data with individual ticks.
-
-Before: All ticks written directly to coinPrices table
-After: Ticks → Redis → Aggregation → Postgres history
-
-Related: #123
-```
-
-## Breaking Change Commits
-
-### API Breaking Change
-```
-feat(api)!: change price endpoint response format [doctrine: Commercial]
-
-BREAKING CHANGE: /api/coins/[id]/price now returns object instead of number
-
-Before: { price: 50000 }
-After: { currentPrice: 50000, timestamp: "...", source: "..." }
-
-Rationale: Commercial Guardrails require timestamp and source tracking.
-
-Migration:
-1. Deploy v2 endpoint at /api/v2/coins/[id]/price
-2. Update all client consumers
-3. Deprecate v1 after 30 days
-4. Remove v1 at day 60
-
-Rollback plan: Revert to commit abc123, redeploy
-
-Ref: COIN-1234
-```
-
-## Doctrine Commits
-
-### Constitutional Enforcement
-```
-doctrine: add validation script for Law-3 compliance
-
-Created scripts/validate_doctrine.js to automatically detect
-realtime injection into charts.
-
-- Pattern matching for WebSocket -> chart flows
-- CI/CD integration ready
-- JSON output for automation
-
-Can be used in pre-commit hooks for enforcement.
-```

package/.agent/skills/git-conventions/metadata/commit-config.yaml

@@ -1,79 +0,0 @@
-# CoinPika Commit Formatter Configuration
-# Defines available commit types, scopes, and doctrines.
-
-commitTypes:
-  - value: feat
-    description: New feature or capability
-  - value: fix
-    description: Bug fix
-  - value: refactor
-    description: Code refactor without behavior change
-  - value: perf
-    description: Performance improvement
-  - value: style
-    description: Code style/formatting (no logic change)
-  - value: docs
-    description: Documentation only
-  - value: test
-    description: Adding or updating tests
-  - value: chore
-    description: Build process, dependencies, tooling
-  - value: doctrine
-    description: Constitutional/doctrinal enforcement
-
-scopes:
-  - value: api
-    description: Backend API
-  - value: data
-    description: Data pipeline or persistence
-  - value: chart
-    description: Chart components
-  - value: mobile
-    description: Mobile UI/gestures
-  - value: worker
-    description: Background jobs/workers
-  - value: cache
-    description: Caching layer
-  - value: auth
-    description: Authentication
-  - value: ui
-    description: General UI components
-
-doctrines:
-  - value: Law-1
-    description: Truth Ownership - Backend owns truth
-  - value: Law-2
-    description: Historical Integrity - Data is immutable
-  - value: Law-3
-    description: Realtime Ephemerality - Separate realtime/historical
-  - value: Law-4
-    description: Chart Truthfulness - Charts from backend only
-  - value: Architecture
-    description: System boundaries
-  - value: Performance
-    description: Caching & optimization
-  - value: Frontend-Mobile
-    description: Mobile UX standards
-  - value: Commercial
-    description: Commercial guardrails
-
-# Maps scopes to suggested doctrines
-scopeMapping:
-  api:
-    - Law-1
-    - Architecture
-    - Performance
-  data:
-    - Law-1
-    - Law-2
-    - Architecture
-  chart:
-    - Law-3
-    - Law-4
-  mobile:
-    - Frontend-Mobile
-  worker:
-    - Performance
-    - Architecture
-  cache:
-    - Performance

package/.agent/skills/git-conventions/metadata/intent-triggers.yaml

@@ -1,25 +0,0 @@
-# Intent Triggers
-# Defines when coinpika-commit-formatter skill should activate
-
-triggers:
-  - pattern: "format.*commit"
-    confidence: high
-  - pattern: "write.*commit.*message"
-    confidence: high
-  - pattern: "commit.*message"
-    confidence: medium
-  - pattern: "conventional.*commit"
-    confidence: high
-  - pattern: "doctrine.*tag"
-    confidence: medium
-
-contexts:
-  - "git_commit"
-  - "pre_commit"
-  - "changelog"
-  - "version_control"
-
-activation_commands:
-  - "git commit"
-  - "format-commit"
-  - "commit-msg"

package/.agent/skills/git-conventions/metadata/scope-mapping.yaml

@@ -1,83 +0,0 @@
-# Scope to Doctrine Mapping
-# Maps commit scopes to suggested doctrine tags
-
-scopes:
-  api:
-    description: Backend API changes
-    doctrines:
-      - Law-1
-      - Architecture
-      - Performance
-    examples:
-      - "feat(api): add caching layer [doctrine: Performance]"
-      - "fix(api): correct price calculation [doctrine: Law-1]"
-
-  data:
-    description: Data pipeline or persistence
-    doctrines:
-      - Law-1
-      - Law-2
-      - Architecture
-    examples:
-      - "feat(data): implement append-only history [doctrine: Law-2]"
-      - "refactor(data): separate realtime buffer [doctrine: Law-3]"
-
-  chart:
-    description: Chart and visualization components
-    doctrines:
-      - Law-3
-      - Law-4
-    examples:
-      - "fix(chart): remove realtime injection [doctrine: Law-3, Law-4]"
-      - "refactor(chart): load from historical API [doctrine: Law-4]"
-
-  mobile:
-    description: Mobile UI and gestures
-    doctrines:
-      - Frontend-Mobile
-    examples:
-      - "feat(mobile): implement swipe tabs [doctrine: Frontend-Mobile]"
-      - "fix(mobile): restore native scroll [doctrine: Frontend-Mobile]"
-
-  worker:
-    description: Background jobs and workers
-    doctrines:
-      - Performance
-      - Architecture
-    examples:
-      - "feat(worker): add price aggregation job [doctrine: Performance]"
-
-  cache:
-    description: Caching layer
-    doctrines:
-      - Performance
-    examples:
-      - "feat(cache): implement Redis caching [doctrine: Performance]"
-
-  auth:
-    description: Authentication
-    doctrines:
-      - Architecture
-    examples:
-      - "feat(auth): add JWT validation [doctrine: Architecture]"
-
-  ui:
-    description: General UI components
-    doctrines: []
-    examples:
-      - "feat(ui): add loading skeleton"
-      - "fix(ui): correct button alignment"
-
-# Types that require doctrine tags
-types_requiring_doctrine:
-  - feat
-  - fix
-  - refactor
-  - perf
-
-# Types that don't need doctrine tags
-types_exempt:
-  - docs
-  - style
-  - test
-  - chore

package/.agent/skills/git-conventions/resources/BEST_PRACTICES.md

@@ -1,121 +0,0 @@
-# Commit Message Best Practices
-
-## CoinPika-Specific Guidelines
-
-### 1. Always Include Context
-
-For fintech applications, context is critical:
-
-```
-# ❌ Bad - No context
-fix: update price
-
-# ✅ Good - Clear context
-fix(api): correct BTC price rounding to 2 decimals [doctrine: Law-1]
-
-Fixes floating-point precision issue that caused 0.01% price drift.
-Affects display only, not stored values.
-```
-
-### 2. Doctrine Tags Are Mandatory
-
-When changing critical areas, ALWAYS include doctrine tags:
-
-| Change Type | Required Tags |
-|-------------|---------------|
-| Price/balance logic | `[doctrine: Law-1]` |
-| Historical data | `[doctrine: Law-2]` |
-| Chart components | `[doctrine: Law-3, Law-4]` |
-| Mobile gestures | `[doctrine: Frontend-Mobile]` |
-| API endpoints | `[doctrine: Architecture]` |
-| Caching | `[doctrine: Performance]` |
-
-### 3. Breaking Changes
-
-Breaking changes in fintech are HIGH RISK:
-
-```
-feat(api)!: change price response format [doctrine: Commercial]
-
-BREAKING CHANGE: /api/coins/[id]/price now returns object
-
-Before: { price: 50000 }
-After: { currentPrice: 50000, timestamp: "...", source: "..." }
-
-Migration:
-1. Deploy v2 endpoint alongside v1
-2. Update all consumers
-3. Deprecate v1 after 30 days
-
-Rollback: Revert to commit abc123
-```
-
-### 4. Document WHY, Not What
-
-```
-# ❌ Bad - Describes WHAT
-refactor(chart): change data loading
-
-# ✅ Good - Explains WHY
-refactor(chart): separate realtime from historical data loading
-
-Historical chart data was contaminated by WebSocket ticks,
-causing visual artifacts during high-frequency trading.
-Realtime price now displays in separate ticker component.
-
-[doctrine: Law-3, Law-4]
-```
-
-## Anti-Patterns to Avoid
-
-### ❌ Vague Messages
-```
-fix: bug
-update: stuff
-changes
-wip
-```
-
-### ❌ Missing Scope for Critical Changes
-```
-feat: add caching  # Which caching? Where?
-```
-
-### ❌ Multiple Unrelated Changes
-```
-feat: add chart, fix login, update deps  # Split into 3 commits
-```
-
-### ❌ Past Tense
-```
-feat(api): added caching  # Should be "add caching"
-```
-
-### ❌ Period at End
-```
-feat(api): add caching.  # No period needed
-```
-
-## Quick Reference
-
-### Commit Types
-| Type | Description | Bump |
-|------|-------------|------|
-| feat | New feature | MINOR |
-| fix | Bug fix | PATCH |
-| perf | Performance | PATCH |
-| refactor | Code restructure | - |
-| docs | Documentation | - |
-| style | Formatting | - |
-| test | Tests | - |
-| chore | Maintenance | - |
-| doctrine | Constitutional | - |
-
-### Commit Template
-```
-<type>(<scope>): <description> [doctrine: <tags>]
-
-<body - explain WHY>
-
-<footer - breaking changes, issue refs>
-```

package/.agent/skills/git-conventions/resources/references.md

@@ -1,53 +0,0 @@
-# References & Resources
-
-## Specifications
-
-### Conventional Commits
-- [Conventional Commits 1.0.0](https://www.conventionalcommits.org/en/v1.0.0/)
-- [Angular Commit Message Format](https://github.com/angular/angular/blob/main/CONTRIBUTING.md#commit)
-- [Commitizen](https://commitizen-tools.github.io/commitizen/)
-
-### Semantic Versioning
-- [Semantic Versioning 2.0.0](https://semver.org/)
-- [npm Semver Calculator](https://semver.npmjs.com/)
-
-## CoinPika Doctrines
-
-### Constitutional Laws
-| Law | Name | Description |
-|-----|------|-------------|
-| Law-1 | Truth Ownership | Backend owns financial truth |
-| Law-2 | Historical Integrity | Historical data is immutable |
-| Law-3 | Realtime Ephemerality | Realtime data is ephemeral |
-| Law-4 | Chart Truthfulness | Charts from backend only |
-
-### Doctrine Files
-- [Master Constitution](../coinpika-doctrine-pack/constitution/coinpika-master-constitution.md)
-- [Architecture Doctrine](../coinpika-doctrine-pack/doctrines/architecture/)
-- [Backend Doctrine](../coinpika-doctrine-pack/doctrines/backend/)
-- [Chart Data Doctrine](../coinpika-doctrine-pack/doctrines/data/)
-- [Mobile Doctrine](../coinpika-doctrine-pack/doctrines/frontend/)
-- [Performance Doctrine](../coinpika-doctrine-pack/doctrines/performance/)
-- [Commercial Doctrine](../coinpika-doctrine-pack/doctrines/commercial/)
-
-## Git Tools
-
-### Commit Helpers
-- [Commitizen CLI](https://github.com/commitizen/cz-cli)
-- [Commitlint](https://commitlint.js.org/)
-- [Husky Git Hooks](https://typicode.github.io/husky/)
-
-### Changelog Generation
-- [Conventional Changelog](https://github.com/conventional-changelog/conventional-changelog)
-- [Release It!](https://github.com/release-it/release-it)
-- [Standard Version](https://github.com/conventional-changelog/standard-version)
-
-## Learning Resources
-
-### Articles
-- [How to Write Good Commit Messages](https://cbea.ms/git-commit/)
-- [A Note About Git Commit Messages](https://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html)
-- [Git Commit Best Practices](https://gist.github.com/luismts/495d982e8c5b1a0ced4a57cf3d93cf60)
-
-### Videos
-- [Conventional Commits Explained](https://www.youtube.com/results?search_query=conventional+commits)