diff --git a/actions/azurestorage.js b/actions/azurestorage.js index 3cc4f2d2..773ad7cb 100644 --- a/actions/azurestorage.js +++ b/actions/azurestorage.js @@ -1299,7 +1299,7 @@ export const getProgressBlobs = async (containerName, caseReference) => { const blobPathParts = blob.name.split("/"); const appealBlobName = blobPathParts[1]; - console.log("getProgressBlobs in here", blobPathParts); + consoleLogger("getProgressBlobs in here", blobPathParts); blobObj.push({ "name": appealBlobName, "path": blob.name, @@ -1483,7 +1483,7 @@ export const getRepsFilesBlobs = async ( .split("/")[2] .slice(0, blob.name.split("/")[2].indexOf("_")); - console.log(blob.name); + consoleLogger(blob.name); blobObj.push({ "name": repFileName, @@ -1514,7 +1514,7 @@ export const getRepsFilesBlobs = async ( repFileName }); } - console.log("blobObj:", blobObj); + consoleLogger("blobObj:", blobObj); return blobObj; }; diff --git a/memory-bank/change-log.md b/memory-bank/change-log.md index d804bdab..5b3884bb 100644 --- a/memory-bank/change-log.md +++ b/memory-bank/change-log.md @@ -3038,3 +3038,36 @@ Validation: Follow-ups: - Optional next bounded slice: logging-only normalization in these same azurestorage functions (no behavior change), done separately from structural refactors. + +--- + +### CL-085: TASK22269 Slice B1.10 — azurestorage touched-function logging normalization + +date: 2026-03-26 +author: Cline +scope: `actions/azurestorage.js` +type: change +rationale: Execute the next bounded, logging-only slice by normalizing selected touched-function logs to `consoleLogger` for consistency with current helper/error logging style. +impact: Observability consistency improvement only; no API/route behavior or payload contract changes. +status: completed + +Summary: + +- In previously touched helper-consumer functions only: + - `getProgressBlobs` + - `getRepsFilesBlobs` +- Replaced selected direct `console.log(...)` calls with `consoleLogger(...)`: + - progress blob path-parts trace + - per-blob name trace in reps file listing + - final `blobObj` trace in reps file listing +- Scope intentionally excludes broader file-wide logging normalization to keep risk bounded. + +Validation: + +- `npm run lint` -> pass with warnings only (pre-existing `react-hooks/exhaustive-deps`; no new lint errors) +- `node tests/phase22/index.test.cjs` -> pass +- `node tests/phase7/service-behaviour.test.cjs` -> pass (13/13) + +Follow-ups: + +- Optional next bounded slice: prune currently-unused local `blobDocumentType` variables in the same touched functions (readability-only, no behavior change).