Skip to content

Commit 0884855

Browse files
rjmurillo-botclaude
andcommitted
fix(memory): enforce lean index format, remove ephemeral cache files
## CRITICAL: Index File Format Index files (skills-*-index.md) MUST contain ONLY the table: - No headers, no descriptions, no metadata - Maximum token efficiency Stripped all 30 index files to table-only format. ## Cache Strategy Update Removed ephemeral cache files from git: - github-open-prs-cache.md (deleted) - github-open-issues-cache.md (deleted) Reason: Cache files in git would cause merge conflicts and slow merge velocity. Recommendation: Use session-local or cloudmcp caching instead. ## Agent Documentation Added CRITICAL guidance to memory.md, skillbook.md, and shared templates about index file format requirements. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 6028d67 commit 0884855

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

42 files changed

+168
-655
lines changed

.serena/memories/github-open-issues-cache.md

Lines changed: 0 additions & 58 deletions
This file was deleted.

.serena/memories/github-open-prs-cache.md

Lines changed: 0 additions & 38 deletions
This file was deleted.

.serena/memories/issue-307-memory-automation.md

Lines changed: 28 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,17 +6,37 @@
66
- **Indexed Files**: 197 atomic skills
77
- **Token Efficiency**: 82% savings with session caching
88

9-
## Proposed Cache-Aside Memory Domains
9+
## Cache Strategy (DECISION NEEDED)
1010

11-
### Priority 1: High-Value Caches (Reduce GitHub API calls)
11+
### Problem
1212

13-
| Domain | Cache Target | TTL | Invalidation Trigger |
14-
|--------|--------------|-----|----------------------|
15-
| `github-open-issues-cache` | Open issues list | 1 hour | Issue created/closed webhook |
16-
| `github-open-prs-cache` | Open PRs list | 30 min | PR opened/merged/closed |
17-
| `github-labels-milestones-cache` | Labels and milestones | 24 hours | Manual or on change |
13+
Ephemeral cache files (open PRs, open issues) in git would:
14+
1. Cause merge conflicts on every PR
15+
2. Slow merge velocity
16+
3. Require constant conflict resolution from main
1817

19-
**Benefit**: Agents frequently query open issues/PRs. Caching saves 10-20 API calls per session.
18+
### Options
19+
20+
| Option | Pros | Cons |
21+
|--------|------|------|
22+
| **Session-local cache** | No merge conflicts | No cross-session benefit |
23+
| **cloudmcp cache** | Cross-session, no conflicts | External dependency |
24+
| **Invalidate-on-write** | Fresh data guaranteed | Requires write hooks |
25+
| **No cache (query API)** | Always fresh | Higher API usage |
26+
27+
### Recommendation
28+
29+
Do NOT store ephemeral cache data in `.serena/memories/`. Use one of:
30+
1. Session-local variables (in-context)
31+
2. cloudmcp for cross-session caching
32+
3. Local temp files outside git tracking
33+
34+
### Invalidation Strategy
35+
36+
Prefer invalidate-on-write over TTL-on-read:
37+
- When closing PR/Issue: invalidate cache
38+
- When creating PR/Issue: invalidate cache
39+
- Avoids stale data from TTL-based reads
2040

2141
### Priority 2: Architecture Reference (Rarely Changes)
2242

.serena/memories/memory-index.md

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -41,15 +41,6 @@
4141
| cva refactoring variant consolidation template generate | skills-cva-refactoring |
4242
| phase3 agent skill session handoff template verification | skills-agent-workflow-phase3 |
4343

44-
## GitHub Cache (Cache-Aside Pattern)
45-
46-
| Cache Keywords | Memory File | TTL |
47-
|----------------|-------------|-----|
48-
| open pr list pull request ready review | github-open-prs-cache | 30 min |
49-
| open issue list bug enhancement priority | github-open-issues-cache | 1 hour |
50-
51-
**Usage**: Read cache first. If stale (check timestamp), refresh from API and update memory.
52-
5344
## User Constraints (MUST READ)
5445

5546
| Constraint | Memory |
Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,3 @@
1-
# Agent Workflow Domain Index
2-
3-
**Purpose**: Route to atomic skills for multi-agent pipelines, scope discipline, and execution patterns.
4-
5-
**Source**: Linting session, PR43 retrospective
6-
7-
## Activation Vocabulary
8-
91
| Keywords | File |
102
|----------|------|
113
| pipeline analyst architect planner critic implementer qa full | agent-workflow-pipeline |
@@ -14,14 +6,3 @@
146
| scope discipline discovery defer document expand mid-execution | agent-workflow-scope-discipline |
157
| MVP ship working perfect gold-plating tech debt constraint | agent-workflow-mvp-shipping |
168
| collaboration user addition scope gap signal learning heuristic | agent-workflow-collaboration |
17-
18-
## Domain Statistics
19-
20-
- **Skills**: 6 core patterns
21-
- **Anti-Patterns**: 3 documented
22-
- **Pipeline Stages**: 7
23-
24-
## See Also
25-
26-
- `skills-orchestration-index` - Parallel execution coordination
27-
- `skills-quality-index` - QA routing patterns
Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,5 @@
1-
# Analysis Domain Index
2-
3-
**Purpose**: Route to atomic skills for capability gap analysis, comprehensive analysis standards, and root cause investigation.
4-
5-
**Source**: `.agents/analysis/` patterns
6-
7-
## Activation Vocabulary
8-
91
| Keywords | File |
102
|----------|------|
113
| gap analysis template severity root cause remediation affected | analysis-gap-template |
124
| comprehensive analysis options trade-off recommendation specs | analysis-comprehensive-standard |
135
| git blame root cause commit PR investigation history context | analysis-git-blame |
14-
15-
## Domain Statistics
16-
17-
- **Skills**: 3 core skills
18-
- **Validation**: Sessions 19-21 (100% implementation accuracy)
19-
20-
## See Also
21-
22-
- `skills-critique` - Conflict escalation
23-
- `skills-planning-index` - Parallel exploration
Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,6 @@
1-
# Architecture Domain Index
2-
3-
**Purpose**: Route to atomic skills for ADRs, tool allocation, model selection, and composite actions.
4-
5-
**Source**: `.agents/architecture/` ADRs
6-
7-
## Activation Vocabulary
8-
91
| Keywords | File |
102
|----------|------|
113
| tool allocation role-specific agent blanket security implementer analyst | architecture-tool-allocation |
124
| model selection opus sonnet haiku tier complexity cost | architecture-model-selection |
135
| composite action reusable workflow DRY extract parameterized | architecture-composite-action |
146
| producer consumer prompt coordination handoff format output | architecture-producer-consumer |
15-
16-
## Domain Statistics
17-
18-
- **Skills**: 4 core skills
19-
- **Sources**: ADR-0002, ADR-0003, Session 03, Session 17
20-
21-
## See Also
22-
23-
- `skills-design-index` - Agent design principles
24-
- `skills-orchestration-index` - Agent coordination patterns
Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,5 @@
1-
# Bash Integration Domain Index
2-
3-
**Purpose**: Route to atomic skills for bash script development, cross-language exit codes, and pre-commit hook patterns.
4-
5-
**Source**: PR #52 retrospective
6-
7-
## Activation Vocabulary
8-
91
| Keywords | File |
102
|----------|------|
113
| bash pattern discovery AUTOFIX pre-commit search grep section | bash-pattern-discovery |
124
| exit code return PowerShell script scope cross-language contract | bash-exit-codes |
135
| LASTEXITCODE pester integration test cross-language invoke | bash-exit-code-testing |
14-
15-
## Domain Statistics
16-
17-
- **Skills**: 5 core skills
18-
- **Key Pattern**: exit 1 (not return) in script scope
19-
20-
## See Also
21-
22-
- `skills-pester-testing` - Pester test patterns
23-
- `git-hook-patterns` - Git hook patterns
24-
- `skills-powershell-index` - PowerShell skills
Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,3 @@
1-
# CI Infrastructure Skills Index
2-
3-
**Domain**: CI/CD pipelines, GitHub Actions, build automation
4-
**Skills**: 20
5-
**Updated**: 2025-12-23
6-
7-
## Activation Vocabulary
8-
91
| Keywords | File |
102
|----------|------|
113
| pester test runner invoke artifacts output xml junit nunit CI local | ci-test-runner-artifacts |
@@ -24,9 +16,6 @@
2416
| integration test workflow dry-run module import validation pre-merge | ci-quality-gates |
2517
| research platform documentation limitation compatibility known issue | ci-deployment-validation |
2618
| label pre-validation create ensure exists gh label create deploy | ci-deployment-validation |
27-
28-
## Coverage
29-
3019
| File | Skills | Focus |
3120
|------|--------|-------|
3221
| ci-test-runner-artifacts | TestRunner-001, Artifacts-001 | Test execution patterns |
@@ -38,9 +27,3 @@
3827
| ci-ai-integration | Structured-Output-001, Comment-001 | AI automation |
3928
| ci-quality-gates | CI-001, Infra-003, Integration-001 | Validation gates |
4029
| ci-deployment-validation | Research-002, Infra-004 | Pre-deployment checks |
41-
42-
## Related
43-
44-
- Source: Sessions 03-08, PRs #47, #79, #202, #212
45-
- GitHub workflow patterns: `skills-github-workflow-patterns`
46-
- PowerShell CI: `powershell-cross-platform-ci`
Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,3 @@
1-
# CodeRabbit Domain Index
2-
3-
**Purpose**: Route to atomic skills for CodeRabbit configuration, false positive patterns, and path-based filtering.
4-
5-
**Consolidated Sources**: coderabbit-config-strategy, coderabbit-path-instructions, coderabbit-false-positives
6-
7-
## Activation Vocabulary
8-
91
| Keywords | File |
102
|----------|------|
113
| config profile chill assertive path-filter noise reduction yaml | coderabbit-config-strategy |
@@ -14,8 +6,3 @@
146
| infrastructure naming spaces security injection vulnerable expression | coderabbit-security-false-positives |
157
| generated files edit warning analyst impact-analysis nested fence backtick | coderabbit-documentation-false-positives |
168
| markdownlint MD004 MD012 MD013 duplicate linting yaml | coderabbit-markdownlint |
17-
18-
## Domain Statistics
19-
20-
- **Skills**: 6 atomic skills
21-
- **Validated**: 12 PRs

0 commit comments

Comments
 (0)