Skip to content

fix: validate knowledge-base inputs during dry-run scans - #284

Merged
kmbroai merged 1 commit into
mainfrom
dev/kyleb/validate-dry-run-knowledge-base
Aug 6, 2026
Merged

fix: validate knowledge-base inputs during dry-run scans#284
kmbroai merged 1 commit into
mainfrom
dev/kyleb/validate-dry-run-knowledge-base

Conversation

@kmbroai

@kmbroai kmbroai commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Validate knowledge-base inputs during SDK preflight so codex-security scan --dry-run rejects the same invalid documents as an actual scan.
  • Reuse existing document parsing and clean up temporary staging immediately without initializing the Codex runtime.
  • Add regression coverage for valid Markdown, missing files, unsupported formats, malformed PDFs, empty directories, and symlinks.

Validation

  • Full SDK test suite: 908 passed, 10 expected platform/integration skips.
  • TypeScript typecheck, generated-model verification, and formatting checks.
  • Built the CLI and executed all five invalid knowledge-base cases plus a valid Markdown case.
  • Packed and installed the npm artifact; package validation passed with 198 archive entries and all 106 bundled plugin files.

Reuse full scan document validation during SDK preflight so dry-run scans reject missing, unsupported, malformed, empty, and symlinked knowledge-base inputs before runtime initialization.
@github-actions github-actions Bot added the bug Something isn't working label Aug 5, 2026
@kmbroai
kmbroai merged commit 442e593 into main Aug 6, 2026
17 checks passed
@kmbroai
kmbroai deleted the dev/kyleb/validate-dry-run-knowledge-base branch August 6, 2026 05:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants