Skip to content

Comments

Fix framework load precedence and gosec failure#10

Merged
davidahmann merged 1 commit intomainfrom
codex/fix-framework-load-precedence
Feb 19, 2026
Merged

Fix framework load precedence and gosec failure#10
davidahmann merged 1 commit intomainfrom
codex/fix-framework-load-precedence

Conversation

@davidahmann
Copy link
Contributor

Summary

  • make framework.Load resolve embedded starter frameworks before probing local files for non-path inputs
  • preserve runtime filesystem loading for explicit paths and non-embedded filename fallback
  • add regression tests to prevent local-file collisions from overriding embedded built-ins
  • annotate LoadFile with a scoped #nosec G304 justification for intentional caller-provided local YAML loading

Why

  • fixes deterministic behavior regression where local eu-ai-act.yaml could affect proof frameworks list
  • fixes gosec failure from run 22164193188

Validation

  • go test ./...
  • go vet ./...
  • gosec ./...

@davidahmann davidahmann merged commit 9f7cd15 into main Feb 19, 2026
11 checks passed
@davidahmann davidahmann deleted the codex/fix-framework-load-precedence branch February 19, 2026 01:06
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.

1 participant