Skip to content

Commit cb1d13b

Browse files
committed
docs(audit): record F-01-F-08 enterprise audit-fix evidence
1 parent f2b1210 commit cb1d13b

3 files changed

Lines changed: 62 additions & 0 deletions

File tree

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
# Enterprise Audit-Fix Classification
2+
3+
Source: `gsd-audit-fix --severity all --max 8`
4+
5+
| ID | Severity | Classification | Finding |
6+
|---|---|---|---|
7+
| F-01 | High | Auto-fixable | Clean clone could not collect the shipped full test suite because required runtime modules and dependency/bootstrap files were missing. |
8+
| F-02 | High | Auto-fixable | CI ran only two static contract tests and could report green while the broader suite was broken. |
9+
| F-03 | High | Auto-fixable | `SessionStore` composed unvalidated session, stage, and attempt identifiers into filesystem paths. |
10+
| F-04 | High | Auto-fixable | Dashboard CORS allowed wildcard origins with credentials. |
11+
| F-05 | Medium | Auto-fixable | Repo write operations accepted arbitrary encodings and used non-atomic direct writes. |
12+
| F-06 | Medium | Auto-fixable | Missing `.gitignore` exposed secrets and runtime/test artifacts to accidental commits. |
13+
| F-07 | Medium | Manual-only | Legacy dashboard and MAF runtime contracts overlap and require an architectural consolidation decision. |
14+
| F-08 | Medium | Manual-only | Production authentication, worker isolation, and remote deployment boundaries remain intentionally unimplemented. |
Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
# Enterprise Audit-Fix Summary
2+
3+
## Result
4+
5+
Six auto-fixable findings were resolved. Two architecture/deployment findings remain manual-only.
6+
7+
| ID | Status | Commit |
8+
|---|---|---|
9+
| F-01 | Fixed | `65cb4f2`, `f2b1210` |
10+
| F-02 | Fixed | `fa78d6b` |
11+
| F-03 | Fixed | `48bdb95` |
12+
| F-04 | Fixed | `92e3adf` |
13+
| F-05 | Fixed | `76eda85` |
14+
| F-06 | Fixed | `4975e0b` |
15+
| F-07 | Manual-only | Architecture decision required |
16+
| F-08 | Manual-only | Production boundary decision required |
17+
18+
## Outcome
19+
20+
- Restored a runnable clean-clone contract with declared dependencies, example configuration, launcher, and required runtime modules.
21+
- Replaced narrow green CI with full Windows/Linux validation.
22+
- Closed durable-state path traversal and wildcard credentialed CORS risks.
23+
- Made agent-driven repo writes atomic and UTF-8-only.
24+
- Protected local secrets and generated state from accidental commits.
25+
- Updated README claims to match the verified repository state.
Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# Enterprise Audit-Fix Verification
2+
3+
## Automated Checks
4+
5+
- `python -m pytest -q --tb=short`: 71 passed, 4 subtests passed
6+
- `python -m compileall autogen_starter autogen_dashboard maf_starter main.py -q`: passed
7+
- `node --check autogen_dashboard/static/app.js`: passed
8+
- `git diff --check`: passed
9+
- `python main.py providers`: launcher and provider inventory passed
10+
- `git check-ignore -v .env .pytest_cache .tmp-tests example.err.log`: expected ignore rules passed
11+
12+
## Environment Note
13+
14+
`python -m pip check` reports an unrelated workstation-level preview dependency mismatch:
15+
`agent-framework-core 1.0.0rc5` expects `azure-ai-projects>=2.0.0,<3.0`, while the shared environment contains `azure-ai-projects 2.0.0b3`.
16+
CI installs from `requirements.txt` in a clean environment and now treats dependency consistency as a required gate.
17+
18+
## Manual Verification Remaining
19+
20+
- Observe the Windows and Linux GitHub Actions jobs on PR #1 after push.
21+
- Exercise one real provider-backed dashboard run with a non-production key.
22+
- Decide whether to consolidate or retire the parallel legacy and MAF runtime contracts.
23+
- Define production authentication and isolated worker execution before any non-loopback deployment.

0 commit comments

Comments
 (0)