Skip to content

Restructure agent instructions and skills#366

Merged
gtsiolis merged 2 commits into
mainfrom
update-agent-config
Jul 20, 2026
Merged

Restructure agent instructions and skills#366
gtsiolis merged 2 commits into
mainfrom
update-agent-config

Conversation

@gtsiolis

@gtsiolis gtsiolis commented Jul 4, 2026

Copy link
Copy Markdown
Member

Motivation

The agent-facing instructions had drifted from the code (nonexistent APIs in skills, an incomplete architecture map) and the root CLAUDE.md had grown past 4k words of always-loaded context, much of it deep per-feature reference.

Changes

  • Root CLAUDE.md goes from 4,235 to 3,158 words (-25%): ~1,650 words of deep reference moved into nested files (loaded only when working in those packages) while ~570 words of new conventions were added
  • Restructure CLAUDE.md: complete the internal/ package map and command groups, correct the test-integration and mock-generation claims (mockgen via go:generate), and add sections for commit/PR/Linear conventions, the release process, spec-driven (openspec) changes, and human-facing writing style, plus notes on the single-test/lint/JUnit workflows
  • Move deep reference sections (snapshots, Azure setup/interception, volume mounts, GATEWAY_LISTEN + offline fallbacks) into nested CLAUDE.md files under internal/{snapshot,azureconfig,config,container}/, each with an AGENTS.md symlink mirroring the root convention
  • Refresh skills: fix add-command (the sink.Emit(output.XxxEvent{...}) pattern instead of nonexistent output.EmitXxx helpers, initConfig(nil), Cobra group registration, requireSubcommand, isInteractiveMode, testEnvWithHome), fix the review-pr events checklist (sealedEvent() marker, no emit helper), update create-pr (current Linear MCP tool name, Linear-first ticket step), and refresh add-component (SuccessColor palette constant)
  • Make test-integration depend on build (one-line Makefile change) so the binary is always rebuilt before integration tests — this is what makes the updated build docs accurate; previously a stale bin/lstk could silently run after source edits

Closes PRO-365

@gtsiolis gtsiolis added semver: patch docs: skip Pull request does not require documentation changes labels Jul 4, 2026
@gtsiolis gtsiolis changed the title Fix stale test-integration binary and restructure agent docs Restructure agent instructions and refresh skills Jul 4, 2026
@gtsiolis gtsiolis self-assigned this Jul 4, 2026
@gtsiolis gtsiolis changed the title Restructure agent instructions and refresh skills Restructure agent instructions and skills Jul 4, 2026
@gtsiolis
gtsiolis force-pushed the update-agent-config branch from 27b958d to 5067f2a Compare July 7, 2026 07:13
@gtsiolis
gtsiolis marked this pull request as ready for review July 7, 2026 17:32
@gtsiolis
gtsiolis requested a review from a team as a code owner July 7, 2026 17:32
@gtsiolis
gtsiolis force-pushed the update-agent-config branch 5 times, most recently from ecfcc09 to d957c29 Compare July 15, 2026 07:10
@gtsiolis
gtsiolis force-pushed the update-agent-config branch from d957c29 to a6e0f07 Compare July 16, 2026 07:07
@gtsiolis
gtsiolis force-pushed the update-agent-config branch from a6e0f07 to 4944c46 Compare July 20, 2026 07:11
@gtsiolis

Copy link
Copy Markdown
Member Author

Thanks for taking a look, @skyrpex! Expecting the separate agent instructions in sub-folders to make better use of context, but let's revert if this does not work as expected. Cc @anisaoshafi

@gtsiolis
gtsiolis merged commit 1f8f017 into main Jul 20, 2026
17 checks passed
@gtsiolis
gtsiolis deleted the update-agent-config branch July 20, 2026 10:02

Copy link
Copy Markdown
Member Author

Thanks for merging, @skyrpex! 🏀


Generated by Claude Code

@skyrpex

skyrpex commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

@gtsiolis I've seen this technique being mentioned a few times as well, also heard it works well hence my approval. Let's keep an eye on it! Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs: skip Pull request does not require documentation changes semver: patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants