chore: cleanup pass 1 — remove PLAN.md, orphan npm artifacts; reconcile docs #1

Merged

millaredos merged 4 commits from feat/cleanup-pass-1 into main

2026-08-22 22:28:45 +00:00

MillasAgent commented

2026-08-22 20:48:55 +00:00

Owner

Cleanup pass: remove PLAN.md + orphan npm artifacts + reconcile docs

Summary

A 4-commit cleanup PR that retires the now-implemented PLAN.md, removes orphan npm artifacts (@opencode-ai/plugin@1.18.21 + transitive deps) that no code path in the bundle consumed, and reconciles the surviving documentation with the actual file tree.

Commits

b46b24b chore: remove PLAN.md (already implemented)
a3f4f99 chore: remove orphan npm artifacts and clean inner .gitignore
be8dac2 docs: reconcile README, CHANGELOG, AGENTS.md with current state
50fb796 docs: address review nits on cleanup-pass-1

Tech-debt closures

TD-1: spec.md reframed as intentional forward-looking spec (license reconciliation deferred to a later PR).
TD-2: inner workspace/opencode/.gitignore self-ignore removed; trimmed to a single node_modules entry.
TD-3: orphan npm artifacts deleted.
TD-4: deployed workspace/opencode/AGENTS.md now listed in README tree.
TD-5: PLAN.md stale frontmatter / counts eliminated.
TD-6: README tree counts accurate (7 agents, 15 commands, 5 skills).
TD-7: README §Quick start documents the Context7 placeholder (config-level fix deferred).
TD-9: tui.json surfaced in README tree; plugin-source decision still pending.
TD-10: README tree now lists opencode.jsonc, tui.json, AGENTS.md, .gitignore, codegraph/.
TD-11: CHANGELOG [0.1.1]: link target added.
TD-15: branch convention feat/<slug> / fix/<slug> added to README §Conventions.

Verification

bash workspace/opencode/deploy.sh --check → exit 0.
grep PLAN.md and grep @opencode-ai/plugin return only the CHANGELOG entries that document the removals (Keep-a-Changelog compliant).
Double-blind review (security-auditor + arch-quality-reviewer in parallel) returned PASS WITH NITS; all raised items addressed in commit 4.

# Cleanup pass: remove PLAN.md + orphan npm artifacts + reconcile docs ## Summary A 4-commit cleanup PR that retires the now-implemented `PLAN.md`, removes orphan npm artifacts (`@opencode-ai/plugin@1.18.21` + transitive deps) that no code path in the bundle consumed, and reconciles the surviving documentation with the actual file tree. ## Commits 1. `b46b24b chore: remove PLAN.md (already implemented)` 2. `a3f4f99 chore: remove orphan npm artifacts and clean inner .gitignore` 3. `be8dac2 docs: reconcile README, CHANGELOG, AGENTS.md with current state` 4. `50fb796 docs: address review nits on cleanup-pass-1` ## Tech-debt closures - **TD-1**: spec.md reframed as intentional forward-looking spec (license reconciliation deferred to a later PR). - **TD-2**: inner `workspace/opencode/.gitignore` self-ignore removed; trimmed to a single `node_modules` entry. - **TD-3**: orphan npm artifacts deleted. - **TD-4**: deployed `workspace/opencode/AGENTS.md` now listed in README tree. - **TD-5**: PLAN.md stale frontmatter / counts eliminated. - **TD-6**: README tree counts accurate (7 agents, 15 commands, 5 skills). - **TD-7**: README §Quick start documents the Context7 placeholder (config-level fix deferred). - **TD-9**: `tui.json` surfaced in README tree; plugin-source decision still pending. - **TD-10**: README tree now lists `opencode.jsonc`, `tui.json`, `AGENTS.md`, `.gitignore`, `codegraph/`. - **TD-11**: CHANGELOG `[0.1.1]:` link target added. - **TD-15**: branch convention `feat/<slug>` / `fix/<slug>` added to README §Conventions. ## Verification - `bash workspace/opencode/deploy.sh --check` → exit 0. - `grep PLAN.md` and `grep @opencode-ai/plugin` return only the CHANGELOG entries that document the removals (Keep-a-Changelog compliant). - Double-blind review (security-auditor + arch-quality-reviewer in parallel) returned PASS WITH NITS; all raised items addressed in commit 4.

MillasAgent added 4 commits

2026-08-22 20:48:55 +00:00

chore: remove PLAN.md (already implemented) b46b24b534

The internal Spanish planning document is no longer needed; its decisions are encoded in AGENTS.md §2, §5 and CHANGELOG.md.

chore: remove orphan npm artifacts and clean inner .gitignore a3f4f9951a

The workspace/opencode/ npm artifacts (@opencode-ai/plugin@1.18.21 + transitive deps) were never consumed by any code path; deploy.sh is pure bash + python3/yq. The inner .gitignore listed entries for the deleted artifacts and self-ignored itself.

docs: reconcile README, CHANGELOG, AGENTS.md with current state be8dac2f06

Documentation reconciled with the post-cleanup state. Resolves TD-4, TD-5, TD-6, TD-10, TD-11, TD-15 (fully) and TD-7, TD-9 (partially, via doc notes). TD-1 reframed: spec.md is an intentional forward-looking spec (license reconciliation deferred).

docs: address review nits on cleanup-pass-1 50fb79681c

Fixes raised by the double-blind review of the prior three commits:

- README §Conventions command bullet now mentions both `orchestrator` and
  `codegraph` (matches AGENTS.md §5.3 and the on-disk reality: 7 workflow +
  8 CPG commands).
- README §Repository structure AGENTS.md annotation corrected: the deployed
  file is Context7 instructions only (not "CodeGraph"); the "(read by /init)"
  hint was misleading.
- AGENTS.md §7 TD-1a and §8 step 6 wording tightened to match the §1 framing
  of `spec.md` as intentional (the "split into a separate repo" branch was
  incompatible with §1).
- README §Quick start Context7 paragraph adds a "do not commit your real key"
  caveat (security reviewer info-level).
- CHANGELOG.md gains a trailing newline.

No behavior change; `deploy.sh --check` semantics unaffected.