Skip to content

Commit 5cb5b82

Browse files
committed
docs: add issue 209 final scan counts
1 parent 71ae062 commit 5cb5b82

1 file changed

Lines changed: 44 additions & 8 deletions

File tree

docs/product-name-audit.md

Lines changed: 44 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -45,11 +45,9 @@ git grep -n -i -E 'kapi|ilchul' -- ':!package-lock.json' ':!node_modules'
4545
## Residual scan after runtime alias removal
4646

4747
```text
48-
find src -iname '*kapi*' -print
49-
src/.kapi
50-
src/.kapi/lanes/kapi-autoresearch
51-
src/.kapi/lanes/kapi-autoresearch/kapi-ux-개선-계획
48+
git ls-files 'src/*kapi*' 'src/**/*kapi*' | sort
5249
src/cli/kapi-review-cli.ts
50+
# count: 1 tracked filename hit
5351
5452
node -e 'const p=require("./package.json"); console.log(JSON.stringify(p.bin,null,2))'
5553
{
@@ -61,14 +59,52 @@ rg -n '"kapi"\\s*:|bin/kapi|npm exec -- kapi|kapi compatibility alias|kapi --hel
6159
test/cli-bin.test.ts:41:test("package metadata exposes ilchul runtime bin without kapi compatibility alias", async () => {
6260
```
6361

64-
Remaining `src/` filename hits are legacy fixture state under `src/.kapi` and the literal `kapi-review` external integration CLI. Runtime package metadata no longer exposes a `kapi` bin, and the old direct `bin/kapi.mjs` shim is removed.
62+
Remaining tracked `src/` filename hits are limited to the literal `kapi-review` external integration CLI. Runtime package metadata no longer exposes a `kapi` bin, and the old direct `bin/kapi.mjs` shim is removed. Local ignored/generated fixture state under `src/.kapi` may exist in developer worktrees, but it is not part of the committed source tree scan.
63+
64+
## Final acceptance scan counts
65+
66+
The final issue #209 residual scans were run against the committed runtime-alias-retirement tree. Active `src/` content counts exclude only ignored/generated fixture state so the evidence is reproducible from a clean checkout:
67+
68+
```text
69+
rg -n -i 'kapi' src --glob '!src/.kapi/**' --glob '!src/.git/**'
70+
# 501 matching lines / 590 matches across 36 active src files
71+
72+
rg -n 'Kapi[A-Z]|\bkapi[A-Z_-]|kapi-' src test docs package.json package-lock.json
73+
# 1307 matching lines / 1491 matches across source, tests, docs, and package metadata
74+
75+
rg -n '\b(class|interface|type|function|const|let|var|enum)\s+[A-Za-z0-9_]*Kapi[A-Za-z0-9_]*|\b(class|interface|type|function|const|let|var|enum)\s+kapi[A-Za-z0-9_]*' src
76+
# no internal reusable declaration-name hits
77+
```
78+
79+
Active `src/` residual category counts:
80+
81+
| Category | Lines | Matches | Files | Why retained |
82+
| --- | ---: | ---: | ---: | --- |
83+
| User-facing compatibility text | 228 | 240 | 25 | Status/help/diagnostic wording still describes the existing Kapi workflow contract exposed to users. |
84+
| Serialized workflow/event contracts | 124 | 147 | 20 | Workflow IDs, event kinds, run-contract source kinds, and persisted state keys remain compatibility contracts. |
85+
| Slash commands / public tool contracts | 99 | 126 | 12 | `/kapi-*`, `kapi_*`, and `@kapi:` surfaces are public contract names pending a separate migration plan. |
86+
| External review integration | 24 | 48 | 5 | `kapi-review`, `kapi-agent`, and `kapi-agent/review` are literal external integration names. |
87+
| Environment/runtime compatibility | 10 | 11 | 7 | `KAPI_*` environment variables remain compatibility inputs for runtime dispatch and tests. |
88+
| Other compatibility literals | 16 | 18 | 10 | Shell labels, fixture command strings, and small compatibility literals that are not reusable implementation identifiers. |
89+
90+
## Issue #209 acceptance mapping
91+
92+
| Acceptance criterion | Status | Evidence |
93+
| --- | --- | --- |
94+
| `src/` contains no `kapi` in filenames except explicitly justified boundary exceptions. | Met | Tracked filename scan has 1 hit: `src/cli/kapi-review-cli.ts` for the external review integration. Ignored/generated local `src/.kapi` fixture state is not part of the committed source tree. |
95+
| Internal reusable exports/types/functions/classes no longer use `Kapi*` / `kapi*` naming. | Met | Declaration-name scan returns no active `src` hits. Remaining exported literals are serialized contracts, event names, or external integration strings. |
96+
| Remaining `kapi` hits in `src/` are limited to documented compatibility contracts, serialized workflow IDs, slash-command literals, or intentionally named external integrations. | Met | Category table above classifies all 501 active `src` matching lines / 590 matches. |
97+
| Public runtime command examples remain `ilchul`; internal implementation names remain generic. | Met | Package bins are `ilchul` and `kapi-review`; runtime internals use `runctl` / `runtime` / `workflow` / `worker` names where reusable. |
98+
| Tests prove package/runtime CLI does not expose a `kapi` runtime alias. | Met | `test/cli-bin.test.ts` covers package-local `npm exec -- kapi --help` failure and isolated-prefix absence of a `kapi` shim. |
99+
| Final scan report includes remaining exception categories and counts. | Met | This section records filename, content, semantic scan, declaration scan, and residual category counts. |
65100

66101
## Remaining justified product-name identifiers
67102

68103
- **External review integration:** `kapi-review`, `kapi-agent`, `kapi-agent/review`.
69-
- **Persisted contracts:** `/kapi-*` slash commands, `kapi-*` workflow IDs, existing serialized state expectations.
70-
- **Storage/config namespace:** `.ilchul`, `~/.ilchul`, `ILCHUL_*`, legacy `.kapi` references.
71-
- **External integration:** `kapi-agent`, `kapi-agent/review`.
104+
- **Persisted contracts:** `/kapi-*` slash commands, `kapi-*` workflow IDs, `kapi.worker.*` / `kapi.pr.*` event names, `kapi-workflow` run-contract source kind, and existing serialized state expectations.
105+
- **Public tool / command contracts:** `kapi_*` tool names and `@kapi:` UI completion syntax.
106+
- **User-facing compatibility text:** status, help, validation, and diagnostic strings that describe the still-public Kapi workflow contract.
107+
- **Storage/config namespace:** `.ilchul`, `~/.ilchul`, `ILCHUL_*`, legacy `.kapi` references, and compatibility `KAPI_*` environment inputs.
72108
- **Documentation/history:** prose explaining migration context or historical behavior.
73109

74110
## Not changed in this slice

0 commit comments

Comments
 (0)