fix(init): squads init works in clean environment#611
Closed
agents-squads[bot] wants to merge 5 commits intodevelopfrom
Closed
fix(init): squads init works in clean environment#611agents-squads[bot] wants to merge 5 commits intodevelopfrom
agents-squads[bot] wants to merge 5 commits intodevelopfrom
Conversation
Context loading changes: - Removed L2 (SQUAD.md body injection) — SQUAD.md is now metadata only for CLI routing (repo, agents, config). Not injected into prompt. - Split old L3 (priorities OR goals) into L2 (priorities.md) and L3 (goals.md) as separate layers loaded independently. - Removed L7 (active-work.md) and L8 (briefs/) from context loading. These files still exist but no longer consume context budget. - Renumbered: L6=feedback, L7=daily-briefing, L8=cross-squad learnings. Role-based access updated: - scanner: L1-L5 (company, priorities, goals, agent, state) - worker/verifier: L1-L6 (+ feedback) - lead/coo: L1-L8 (+ daily briefing + cross-squad) Role resolution: - Direct match for new schema (role: "lead" → lead, no scoring needed) - Falls back to token scoring for legacy free-text roles Agent prompt updated to reflect new layer names. Co-Authored-By: Claude <noreply@anthropic.com>
company.md, priorities.md, goals.md, state.md all have YAML frontmatter for CLI metadata. LLMs don't need it — strip before injecting into prompt. Saves ~80 tokens per run. Also: DRYRUN_CONTEXT_MAX_CHARS now configurable via env var SQUADS_DRYRUN_MAX_CHARS for debugging full context output. Co-Authored-By: Claude <noreply@anthropic.com>
New commands: - squads catalog list — show all services grouped by type - squads catalog show <service> — detailed service view - squads catalog check [service] — run scorecard checks (all or one) - squads release pre-check <service> — validate dependencies before deploy New lib modules: - lib/idp/types.ts — TypeScript interfaces matching IDP YAML schema - lib/idp/resolver.ts — find IDP directory (env var → co-located → sibling → absolute) - lib/idp/catalog-loader.ts — parse YAML catalog entries via gray-matter - lib/idp/scorecard-engine.ts — evaluate services against quality checks Scorecard sources: local filesystem, gh CLI, git log. Graceful degradation when gh is unavailable (shows "unknown" vs failing). No new dependencies — YAML parsed via gray-matter's engine. Co-Authored-By: Claude <noreply@anthropic.com>
Dockerfile.fresh-user: clean Node 22 container, npm install -g squads-cli, empty git repo. No config, no .agents, nothing. test-fresh-user.sh: 9-step automated test suite covering the complete first-run flow (version, help, init, status, list, catalog, doctor, unknown command). Current results: 4/9 pass. squads init is broken (#610). Usage: ./test/docker/test-fresh-user.sh --auto # automated ./test/docker/test-fresh-user.sh # interactive Co-Authored-By: Claude <noreply@anthropic.com>
…steps pass - setup-checks.ts: provider CLI missing is now a warning, not a blocking error; users can scaffold first and install the provider CLI later - cli.ts: restore squads list as alias for squads status (was removed/exit-1) - Dockerfile.fresh-user: build from local source (npm pack + install) instead of npm registry, so tests exercise the current codebase - .dockerignore: exclude node_modules/dist to speed up Docker builds Closes #610
Contributor
|
PR #609 (context system + IDP) is now merged to develop. This PR needs a rebase onto develop to pick up the CatalogEntry type. Rebase and re-push to fix the build. |
Contributor
|
Superseded by clean rebase on solve/issue-610-v2 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
squads initwas blocking on missing provider CLI (status: missing→hasErrors = true) — now treated as a warning so users can scaffold first and install latersquads listwas removed (exit 1) — restored as alias forsquads statusDockerfile.fresh-userwas installing from npm registry (old version) — now builds from local source vianpm packso tests exercise current codebase.dockerignoreto excludenode_modules/distfor fast Docker buildsTesting
test/docker/test-fresh-user.sh --autonow pass (was 4/9)Closes #610