forked from tinyhumansai/openhuman
-
Notifications
You must be signed in to change notification settings - Fork 0
Move redux-logger and debug to devDependencies #5
Copy link
Copy link
Closed
Labels
securitySecurity, encryption, approvals, credentials, and trust boundaries.Security, encryption, approvals, credentials, and trust boundaries.subtaskSubtask of a larger tracked effort.Subtask of a larger tracked effort.taskWork item that is not primarily a bug or a feature.Work item that is not primarily a bug or a feature.
Metadata
Metadata
Assignees
Labels
securitySecurity, encryption, approvals, credentials, and trust boundaries.Security, encryption, approvals, credentials, and trust boundaries.subtaskSubtask of a larger tracked effort.Subtask of a larger tracked effort.taskWork item that is not primarily a bug or a feature.Work item that is not primarily a bug or a feature.
Summary
app/package.jsonlistsredux-loggeranddebugindependencies. Move them todevDependenciesand guard usage so production Tauri builds cannot log JWTs / API responses / chat messages.Problem / Context
Confirmed in
app/package.json:"debug": "^4.4.3"underdependencies"redux-logger": "^3.0.6"underdependenciesredux-loggerlogs every Redux action + state change;debugexposes namespaced console output. Even with Vite tree-shaking, runtime-deps placement is a correctness/audit smell. CWE-532.Identified by audit tinyhumansai#2575 (M-2).
Scope
Move both to
devDependencies.Wrap any
redux-loggermiddleware import behindimport.meta.env.DEV:Verify the production bundle (
pnpm build, then grepapp/dist) no longer references either package.Acceptance criteria
redux-logger+debugindevDependenciesonlyRelated