Skip to content

Enforce FR modification notices and fix Bedrock optional dependency handling - #27

Merged
manas-fortifyroot merged 1 commit into
fr-v0.52.6.xfrom
feature/fr-modification-notices-and-bedrock-optional-deps
Jun 2, 2026
Merged

Enforce FR modification notices and fix Bedrock optional dependency handling#27
manas-fortifyroot merged 1 commit into
fr-v0.52.6.xfrom
feature/fr-modification-notices-and-bedrock-optional-deps

Conversation

@manas-fortifyroot

Copy link
Copy Markdown

Summary

Adds a repo-local FR modification notice workflow for upstream-derived files and fixes Bedrock instrumentation so SDK initialization does not hard-crash when boto3 is installed but anthropic is absent.

Changes

  • Added scripts/apply-fr-modification-notices.py with apply/check modes.
  • Added notice normalization coverage for modified upstream .py and .toml files.
  • Added CI enforcement in .github/workflows/fr-pr.yaml.
  • Standardized FR modification notices across upstream-derived modified files.
  • Skips FR-added files from notice insertion.
  • Made Bedrock anthropic handling lazy/guarded.
  • Added regression coverage for Bedrock optional dependency behavior.
  • Rebranded stale user-facing Traceloop guidance in fork runtime code.

Validation

  • python3 scripts/test-apply-fr-modification-notices.py
  • python3 scripts/apply-fr-modification-notices.py --check --base-ref fr-v0.52.6.x
  • Bedrock optional dependency targeted tests.
  • fr-openllmetry-py/scripts/run-tests.sh --all

Notes

This fork should be merged first, then a new fork tag should be cut. The FR SDK should be re-vendored from that final tag before merging the SDK PR.

@manas-fortifyroot
manas-fortifyroot merged commit c1f10de into fr-v0.52.6.x Jun 2, 2026
10 checks passed
@manas-fortifyroot
manas-fortifyroot deleted the feature/fr-modification-notices-and-bedrock-optional-deps branch June 2, 2026 16:26
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