Skip to content

Commit e3370a0

Browse files
committed
chore: update progress for US-079
1 parent cf4e78c commit e3370a0

1 file changed

Lines changed: 15 additions & 0 deletions

File tree

progress.txt

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1237,3 +1237,18 @@ PRD: ralph/kernel-hardening (46 stories)
12371237
- readFile for /dev/stdin, /dev/stdout, /dev/stderr passes through to backing VFS (not intercepted)
12381238
- TestFileSystem supports these passthrough operations for testing
12391239
---
1240+
1241+
## 2026-03-17 - US-079
1242+
- Implemented write-side fs permission denial tests and custom checker tests
1243+
- Fixed core `checkPermission` to propagate `reason` from permission decision to error message
1244+
- Updated `createEaccesError` to accept optional `reason` parameter
1245+
- Updated all `onDenied` callbacks in wrapFileSystem, wrapNetworkAdapter, wrapCommandExecutor, envAccessAllowed
1246+
- Files changed:
1247+
- packages/secure-exec-core/src/shared/errors.ts (reason param on createEaccesError)
1248+
- packages/secure-exec-core/src/shared/permissions.ts (reason propagation in checkPermission + all callbacks)
1249+
- packages/secure-exec/tests/permissions.test.ts (5 new tests)
1250+
- **Learnings for future iterations:**
1251+
- Core `checkPermission` and kernel `checkPermission` had diverged — kernel propagated reason, core didn't
1252+
- fsOpToSyscall maps: write→"write", createDir→"mkdir", rm→"unlink" — match these in expectEacces assertions
1253+
- wrapCommandExecutor passes `cwd: options.cwd` to the childProcess permission checker request
1254+
---

0 commit comments

Comments
 (0)