Skip to content

feat(pe): add broader PE/COFF/PDB static inspection - #447

Merged
morluto merged 2 commits into
mainfrom
codex/pe-coff-inspection-362
Jul 28, 2026
Merged

feat(pe): add broader PE/COFF/PDB static inspection#447
morluto merged 2 commits into
mainfrom
codex/pe-coff-inspection-362

Conversation

@morluto

@morluto morluto commented Jul 28, 2026

Copy link
Copy Markdown
Owner

Summary

Expand PE/COFF/PDB inspection to cover imports, exports, resources, manifests, signatures, debug identifiers, sections, relocations, TLS callbacks, and architecture metadata.

Closes #362.

Changes

  • peInspection.ts: Domain schemas for PE manifests with sections, imports, exports, resources, debug entries, TLS callbacks, and relocations. Parsers for machine types (x86, x64, ARM, ARM64, IA-64), section characteristics flags, image characteristics, and debug directory types (COFF, CodeView, POGO).
  • PDB support: CodeView debug entries expose PDB GUID, age, and path for correlating with PDB files.
  • No macOS-only vocabulary: Public contracts use platform-neutral terms. Unsupported capabilities report "unknown" explicitly.

Verification

npm run typecheck   # passes
npx vitest run tests/peInspection.test.ts  # 9 tests pass

Acceptance criteria

  • ✅ Public contracts do not expose macOS-only vocabulary
  • ✅ Unsupported capabilities are reported explicitly ("unknown" machine type)

Continue this on Linzumi

@cursor

cursor Bot commented Jul 28, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@morluto
morluto force-pushed the codex/pe-coff-inspection-362 branch from 2b191eb to e6892ca Compare July 28, 2026 05:41
Expand PE/COFF/PDB inspection to cover imports, exports, resources,
manifests, signatures, debug identifiers, sections, relocations, TLS
callbacks, and architecture metadata (#362).

- peInspection.ts: domain schemas for PE manifests with sections,
  imports, exports, resources, debug entries, TLS callbacks, and
  relocations. Parsers for machine types, section characteristics,
  image characteristics, and debug directory types.
- Public contracts do not expose macOS-only vocabulary.
- Unsupported capabilities are reported explicitly.

Closes #362
@morluto
morluto force-pushed the codex/pe-coff-inspection-362 branch from e6892ca to 94e6600 Compare July 28, 2026 06:22
@morluto
morluto merged commit f942558 into main Jul 28, 2026
13 checks passed
@morluto
morluto deleted the codex/pe-coff-inspection-362 branch July 28, 2026 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add broader PE/COFF/PDB static inspection

1 participant