@agentskillkit/agent-skills 1.0.1

package/.agent/skills/git-conventions/.skill-source.json

@@ -0,0 +1,8 @@
+{
+  "repo": "dataguruin/agent-skills",
+  "skill": "coinpika-commit-formatter",
+  "ref": null,
+  "checksum": "4f9a77947e3ab0f0a89aba0aaeadfceb926cb7e2a48018a624e34cde383b8d29",
+  "installedAt": "2026-01-24T11:02:40.450Z",
+  "method": "symlink"
+}

package/.agent/skills/git-conventions/SKILL.md

@@ -0,0 +1,28 @@
+---
+name: git-conventions
+version: 1.0.0
+description: Enforce Conventional Commits and traceability between code and requirements.
+allowed-tools: Read, Write, Edit, Glob, Grep, Bash
+---
+
+# Git Conventions
+
+> **Philosophy:** A commit message is a love letter to the future maintainer.
+
+## 📚 Atomic Rules (Knowledge)
+
+| Rule | Description |
+|------|-------------|
+| [rules/commit-format.md](rules/commit-format.md) | Conventional Commits structure |
+| [rules/traceability.md](rules/traceability.md) | Linking commits to requirements/docs |
+
+## 🔧 Scripts (Execution)
+
+| Script | Purpose |
+|--------|---------|
+| `scripts/commit_msg.py` | Commit message hook validator |
+
+## ⚠️ Critical Anti-Patterns
+
+- **"WIP" Commits**: Do not push commits named just "WIP" or "fix".
+- **No Scope**: Always provide a scope (e.g., `feat(auth):`).

package/.agent/skills/git-conventions/enforcement/commit-validation-rules.md

@@ -0,0 +1,55 @@
+# Commit Validation Rules
+
+## Overview
+These rules enforce commit message standards for CoinPika codebase.
+
+## Mandatory Rules
+
+### 1. Conventional Commits Format
+All commits MUST follow Conventional Commits specification:
+
+```
+<type>[optional scope]: <description>
+
+[optional body]
+
+[optional footer(s)]
+```
+
+### 2. Doctrine Tagging
+Commits touching the following areas MUST include doctrine tags:
+
+| Scope/Area | Required Tags |
+|------------|---------------|
+| api/, server/ | Law-1, Architecture |
+| chart, graph | Law-3, Law-4 |
+| mobile, gesture | Frontend-Mobile |
+| data, price | Law-1, Law-2 |
+
+Format: `[doctrine: Tag1, Tag2]`
+
+### 3. Breaking Changes
+Breaking changes MUST:
+- Include `!` after type/scope
+- Include `BREAKING CHANGE:` in footer
+- Document rollback plan
+
+## Validation Checklist
+
+- [ ] Type is valid (feat, fix, refactor, etc.)
+- [ ] Description is imperative mood, lowercase
+- [ ] Doctrine tags present if required
+- [ ] Body explains WHY, not what
+- [ ] Breaking changes properly marked
+
+## Enforcement
+
+Run validation:
+```bash
+bash scripts/validate-commit-msg.sh .git/COMMIT_EDITMSG
+```
+
+Or use interactive formatter:
+```bash
+node scripts/format-commit.js
+```

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

@@ -0,0 +1,201 @@
+# 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

@@ -0,0 +1,135 @@
+# 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

@@ -0,0 +1,79 @@
+# 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

@@ -0,0 +1,25 @@
+# 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

@@ -0,0 +1,83 @@
+# 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