Description
The README's own Project Structure comments label js/utils/aiMock.js as "The original mock UI extraction hook (deprecated)", while js/utils/api.js is described as "the live fetch logic communicating with our Express API" — i.e., the actual one in use. Leaving a clearly-marked-deprecated file in the active js/utils/ directory risks contributors importing the wrong module, especially newcomers unfamiliar with the project's history.
Proposed Scope
- Confirm
aiMock.js has zero remaining imports across app.js/store.js
- If unused: delete the file entirely (git history preserves it if ever needed)
- If still referenced anywhere (e.g. as an offline/demo fallback): rename it clearly (e.g.
aiMockFallback.js) and document its actual current purpose in a code comment at the top of the file
- Update the README's Project Structure section accordingly
Acceptance Criteria
Labels: housekeeping, good first issue
Description
The README's own Project Structure comments label
js/utils/aiMock.jsas "The original mock UI extraction hook (deprecated)", whilejs/utils/api.jsis described as "the live fetch logic communicating with our Express API" — i.e., the actual one in use. Leaving a clearly-marked-deprecated file in the activejs/utils/directory risks contributors importing the wrong module, especially newcomers unfamiliar with the project's history.Proposed Scope
aiMock.jshas zero remaining imports acrossapp.js/store.jsaiMockFallback.js) and document its actual current purpose in a code comment at the top of the fileAcceptance Criteria
aiMock.jsis either removed or clearly justified and renamedapp.jsLabels:
housekeeping,good first issue