refactor(cmd): namespace command internals (#550) #554

Merged

barrettruth merged 1 commit from refactor/cmd-namespace-550 into refactor/public-submodule-boundaries-549

2026-04-29 20:03:45 +00:00

barrettruth commented

2026-04-29 20:03:37 +00:00

Problem

The :Forge command implementation still exposed a flat set of internal cmd_* modules at the root of lua/forge/, even though they form one cohesive command subsystem with a single public facade at require('forge.cmd').

Solution

Move the parser, completion, dispatch, and resolver helpers under lua/forge/cmd/, keep require('forge.cmd') stable via cmd/init.lua, and update the command spec harness to preload and reset the new namespaced internal modules.

Issue: #550

## Problem The `:Forge` command implementation still exposed a flat set of internal `cmd_*` modules at the root of `lua/forge/`, even though they form one cohesive command subsystem with a single public facade at `require('forge.cmd')`. ## Solution Move the parser, completion, dispatch, and resolver helpers under `lua/forge/cmd/`, keep `require('forge.cmd')` stable via `cmd/init.lua`, and update the command spec harness to preload and reset the new namespaced internal modules. Issue: #550