anastasiyaw/codex-claude-code-config/architecture-first

Claude Code and agent skill by anastasiyaw/codex-claude-code-config

Trust tier 2
  • community
LLM review pending
Source check pending

Checking that the SKILL.md file is still in the repository.

Decide module boundaries before the first file: what modules exist, which way dependencies point, who owns state, and what each module may know. Use when starting a project, service, site, API, or subsystem; adding a feature with no obvious home; resolving a circular import or inverted framework dependency; or writing an ARCHITECTURE.md or ADR. Do not use for a one-file script, throwaway experiment, bug fix inside an established seam, naming/function-shape cleanup (use code-complexity), an existing oversized module (use refactoring-safely), or capacity/data scaling decisions (use system-and-data-design). This defines earned boundaries; it does not license speculative layers.

Install

npx vskill@latest install anastasiyaw/codex-claude-code-config/architecture-first

SKILL.md

Architecture first — the shape before the first file

Most bad structure is not a bad decision. It is an absent one: code goes where the smallest diff puts it, and the smallest diff is always "next to the last thing". This skill exists to make the layout an explicit, cheap, early decision.

Scope guard — read first

Match the ceremony to the problem. Over-applying this is its own failure mode.

SituationWhat this skill asks of you
Script, spike, one file, throwawayNothing. Skip.
One module, <500 lines, one reason to changeName the module and its one job. Stop.
Service / site / API, several concernsThe full pre-code checklist below.
Multiple teams or deployables, shared domainChecklist + bounded-context map + one ADR per boundary
Multi-stage work or external release prerequisiteChecklist + a small stage map before the first implementation boundary

The one law

Dependencies point inward, toward policy. Business rules must not import the web framework, the ORM, the queue, or the file layout. The reverse is required.

Violating it silently is the failure — a violation that is written down, with the reason and the cost, is a decision. A violation nobody named is erosion.

Practical test: could this module be exercised by a test with no network, no database and no framework? If not, something outer leaked inward.

Pre-code checklist — before the first file

  1. Name the modules by reason to change, not by technical layer. queue, billing,

catalog — not controllers, models, utils. A module that changes for two unrelated reasons is two modules.

  1. Say what each module owns. Especially state: every piece of mutable state has

exactly one owning module, and everyone else asks that module. Module-level mutable state shared across features is the coupling that later makes a split expensive.

  1. Draw the dependency arrows. Any cycle is a design bug, not a build inconvenience.

Any arrow from domain to framework is inverted — fix it with an interface owned by the inner side.

  1. Establish the ubiquitous language. One term, one meaning, in code and in speech.

If the same word means different things in two places, you have found a bounded context boundary — draw it there.

  1. Define the aggregates. What must be consistent in one step, and what may lag.

Transaction boundaries follow this, not the other way round.

  1. Write one vertical slice end-to-end — UI to store to test — before broadening.

A slice that works proves the seams; six half-built layers prove nothing.

  1. Record it. One page: modules, ownership, data flow, external systems. Plus one

short ADR per decision that was genuinely a choice (context, options, decision, consequences). Both live in git, next to the code.

  1. Name the promotion boundaries. When one verified result becomes the input to

another stage, name its contract, inputs, output, and invalidation keys before implementation. A missing signer, VM, account, or remote service is a future BLOCKED stage, not a reason to keep reopening already-proven code.

Stage contracts - when proof becomes an input

For multi-stage work, architecture includes delivery boundaries as well as module boundaries. Keep these states separate:

  • VERIFIED: the scoped behavior passed at one exact revision.
  • SEALED: the verified scope has an immutable receipt and may be consumed by a

following stage.

  • BLOCKED: a named external prerequisite is absent; upstream proof remains valid.
  • SUPERSEDED: a contract, source, or input digest changed, so a successor must be

verified instead of editing history.

The stage map is deliberately smaller than a release plan. For each boundary, name the owning scope, frozen contract, inputs, output, and what invalidates it. Use the machine-readable ledger only when there is a real hand-off between stages: ../proof-verify/references/proven-stage-contracts.md. Do not add it to a one-file change merely because the word "stage" exists.

Review checklist — once code exists

  • Does any inner module import an outer one? Name it or fix it.
  • Is there module-level mutable state touched by more than one feature?
  • Does one file hold routes/handlers for more than one reason to change?
  • Are there two names for the same concept, or one name for two?
  • Can each module's tests run without the framework?
  • Does the ARCHITECTURE.md still describe what is actually there?

Fast decision table

QuestionDefault answer
Layer folders or feature folders?Feature (vertical slices). Layer folders scatter one change across four directories.
Where does validation live?Input shape at the edge; business rules inside. Never only at the edge.
Interface for a single implementation?No — until a second caller or a test double actually needs it.
Microservices?Not yet. Modular monolith with real boundaries first; extract when a module needs its own deploy or scaling.
Where does the ORM model live?Outer. The domain object is not the row.
Shared "utils" module?A smell. Utils is where things go when nobody decided; name the reason instead.

References — load on demand

  • references/clean-architecture/boundaries-and-layers.md — boundaries, Humble Object
  • references/clean-architecture/solid-and-components.md — SOLID applied correctly, REP/CCP/CRP, ADP/SDP/SAP
  • references/clean-architecture/details-and-code-organization.md — DB/web/frameworks as details
  • references/clean-architecture/python-implementation.md — entities, use cases, repositories, wiring
  • references/domain-driven-design/bounded-contexts.md — context mapping
  • references/domain-driven-design/ubiquitous-language.md — one term, one meaning
  • references/domain-driven-design/building-blocks.md — entities, value objects, aggregates
  • references/domain-driven-design/domain-events.md, repositories-factories.md, strategic-design.md
  • *-original.md — the source skills' own framework prose, kept verbatim

Gotchas

  • "We'll structure it later." Later costs more per caller, and callers only grow. The

cheapest moment is before the first file; the second cheapest is now.

  • Ceremony as architecture. Four layers around a CRUD endpoint is not architecture,

it is cost. The scope guard above exists to stop this.

  • Folders instead of boundaries. Moving files without changing who imports whom

changes nothing. The arrows are the architecture; folders only display it.

  • The domain importing the framework "just for a type". That is the whole violation,

arriving politely.

Troubleshooting

SymptomCauseFix
Circular importTwo modules both own part of one conceptExtract the shared concept into a third module both depend on
One change touches six files across four foldersLayer folders, not feature foldersRe-cut by feature; keep the change local
Tests need a live database to assert a ruleRule lives outside the domainMove the rule inward, inject the store
Nobody can say which module owns XNobody decidedDecide now, write it in ARCHITECTURE.md, move the state

Security

  • Security scanPassed, 100 of 100Pattern checks on the skill's files.
  • Model reviewNot runA model review is scheduled for the next rescan.

Quality evaluation

  • Task evaluationNot runNo evaluation yet. Run one in Skill Studio.

Source

Popularity

Repository stars
150
Repository forks
24
7-day trend score
25

Stars and forks are for the whole GitHub repository, not this skill alone. Not refreshed yet.

Works with

Any agent vskill installs to: 56 targets in all. These 39 have a page on this site.

  • Claude Code
  • Cursor
  • GitHub Copilot
  • Windsurf
  • Codex
  • Gemini CLI
  • Cline
  • Amp
  • Roo Code
  • Goose
  • Continue
  • Kilo Code
  • Junie
  • Trae
  • OpenHands
  • Qwen Code
  • AI Agent
  • Tabnine Chat
  • Sourcegraph Cody
  • Amazon Q
  • Firebase Genkit
  • Sweep
  • Devin
  • Phind
  • Mentat
  • Claude Desktop
  • Aider
  • GPT Pilot
  • Plandex
  • bolt.new
  • v0
  • Replit Agent
  • Anse
  • TypingMind
  • BetterChatGPT
  • LibreChat
  • Warp AI
  • Pieces
  • Fig AI
Embed badge
vskill VERIFIED badge
[![vskill VERIFIED](https://verified-skill.com/api/v1/skills/anastasiyaw/codex-claude-code-config/architecture-first/badge)](https://verified-skill.com/skills/anastasiyaw/codex-claude-code-config/architecture-first)

Something wrong with this skill or its page? Tell us.