Skip to content

Commit 108e610

Browse files
authored
Update vulnerable frontend dependencies (#26)
* fix: update vulnerable frontend dependencies * docs: record frontend dependency remediation --------- Co-authored-by: fengting124 <fengting124@users.noreply.github.com>
1 parent 53aa797 commit 108e610

2 files changed

Lines changed: 270 additions & 217 deletions

File tree

docs/project-worklog.md

Lines changed: 45 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -705,24 +705,59 @@ Deferred:
705705

706706
---
707707

708+
### 2026-07-11: Frontend Dependency Security
709+
710+
```text
711+
fix/frontend-dependency-security
712+
```
713+
714+
What changed:
715+
716+
- Refreshed the lockfile within existing package ranges.
717+
- Upgraded Vite from 8.0.14 to 8.1.4 to resolve the Windows path-deny bypass
718+
and UNC-path credential disclosure advisories.
719+
- Upgraded transitive Babel packages from 7.29.0 to 7.29.7 to resolve the
720+
source-map local file-read advisory.
721+
- Kept `package.json` ranges and application source unchanged.
722+
723+
Why:
724+
725+
- A known high-severity development-server vulnerability is still relevant on
726+
the project's primary Windows development environment.
727+
- A lockfile-only compatible update is lower risk than carrying known findings
728+
or mixing a framework migration with security remediation.
729+
730+
Verification:
731+
732+
- A clean `npm ci` completed successfully.
733+
- `npm audit --audit-level=low` reports zero vulnerabilities.
734+
- Frontend tests (8), lint, and production build pass with Vite 8.1.4.
735+
- Screenshots were not repeated because no frontend source, CSS, asset, or
736+
runtime behavior changed.
737+
738+
---
739+
708740
## Next Recommended Work
709741

710-
Resolve the recorded frontend dependency findings in an isolated branch:
742+
Add interview-visible operational observability:
711743

712744
```text
713-
fix/frontend-dependency-security
745+
feature/observability-correlation
714746
```
715747

716748
Scope:
717749

718-
- Re-run `npm audit` against the current lockfile.
719-
- Upgrade only affected direct dependencies within compatible ranges where
720-
possible.
721-
- Verify tests, lint, production build, and UI screenshots.
722-
- Record any finding that requires a breaking framework upgrade.
750+
- Propagate a correlation id across HTTP requests, outbox events, Redis jobs,
751+
model calls, and persisted execution records.
752+
- Add Micrometer counters and timers for dispatch, retries, queue outcomes,
753+
inference latency, and upload rejection reasons.
754+
- Define structured logging fields without logging image bytes or model raw
755+
payloads.
756+
- Document local metrics endpoints and production exposure boundaries.
723757

724758
Reason:
725759

726-
The production foundation audit recorded one high and one low frontend
727-
dependency finding. Closing known supply-chain findings is a smaller but real
728-
production requirement and should not be mixed into backend security code.
760+
The project now handles failure and recovery, but operators cannot yet answer
761+
which request produced a task, where latency accumulated, or how often retries
762+
and security rejections occur. Correlated traces and bounded metrics turn the
763+
reliability features into an operable system.

0 commit comments

Comments
 (0)