Skip to content

fix(xtest)!: remove platform feature overrides - #618

Open
c-r33d wants to merge 2 commits into
mainfrom
codex/remove-platform-force-support
Open

c-r33d wants to merge 2 commits into
mainfrom
codex/remove-platform-force-support

Conversation

@c-r33d

@c-r33d c-r33d commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Remove force-platform-supports / XT_FORCE_PLATFORM_SUPPORTS and restore the SDK override implementation from before #604, including removal of the platform-only validation added in #614. Preserve the KAO tests, fixtures, and KM3 setup.

Merge after #617 enables KAO by platform version. The caller input has already been removed from opentdf/platform#4056.

Validation: 208 offline tests, Ruff lint/format, Pyright, and workflow YAML checks passed; #617 applies cleanly. Full platform integration was not rerun.

Signed-off-by: Chris Reed <creed@virtru.com>
@coderabbitai

coderabbitai Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 37 seconds.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 9904906b-c089-415a-9bcd-484eb99aeca6

📥 Commits

Reviewing files that changed from the base of the PR and between 384b310 and 5ae7a4d.

📒 Files selected for processing (6)
  • .github/workflows/xtest.yml
  • AGENTS.md
  • xtest/AGENTS.md
  • xtest/README.md
  • xtest/tdfs.py
  • xtest/test_tdfs_units.py

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

Signed-off-by: Chris Reed <creed@virtru.com>
@sonarqubecloud

Copy link
Copy Markdown

@c-r33d
c-r33d marked this pull request as ready for review September 22, 2026 19:00
@c-r33d
c-r33d requested review from a team as code owners September 22, 2026 19:00
@c-r33d c-r33d changed the title refactor(xtest)!: remove platform feature overrides fix(xtest)!: remove platform feature overrides Sep 22, 2026
@github-actions

Copy link
Copy Markdown

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