TASK22269: normalize touched azurestorage logging calls
This commit is contained in:
@@ -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;
|
||||
};
|
||||
|
||||
@@ -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).
|
||||
|
||||
Reference in New Issue
Block a user