fix(xtest): detect java dpop support via supports subcommand

[dmihalcik-virtru]

Problem

The java-sdk removed the user-facing --dpop / --dpop-key CLI flags in opentdf/java-sdk#374. DPoP is now always-on by default, and capability is exposed through the supports subcommand instead.

The Java wrapper's supports dpop case still detects DPoP by grepping help encrypt for the --dpop flag:

dpop)
  set -o pipefail
  java -jar "$SCRIPT_DIR"/cmdline.jar help encrypt | grep -iE -- '--dpop'
  exit $?
  ;;

With the flags gone this grep finds nothing, exits 1, and xtest skips java DPoP tests:

SKIPPED [2] tdfs.py:619: java@... sdk doesn't yet support [dpop]

The sibling dpop_nonce_challenge case already delegates to the supports subcommand and is unaffected.

Fix

Detect dpop the same way dpop_nonce_challenge already does — via the supports subcommand, whose exit code is the capability contract (0 = supported, 1 = not):

dpop)
  java -jar "$SCRIPT_DIR"/cmdline.jar supports dpop
  exit $?
  ;;

set -o pipefail is dropped since there's no longer a pipe.

Verification

With a current java cmdline.jar installed: ./cli.sh supports dpop; echo $? prints 0, and dpop_nonce_challenge still returns 0. Only xtest/sdk/java/cli.sh changes; other SDK wrappers are untouched.

Summary by CodeRabbit

• Bug Fixes
  • Improved detection of DPoP support in the Java CLI feature probe.
  • Prevents incorrect results caused by relying on help output and flag matching.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[gemini-code-assist]

Code Review

This pull request simplifies the 'dpop' support check in the xtest/sdk/java/cli.sh script. It replaces a pipeline that grepped the help output of the encrypt command with a direct call to supports dpop on the Java command-line utility. There are no review comments, and I have no additional feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 41c6e600-0cb2-4dbf-ad95-043b83024161

📥 Commits

Reviewing files that changed from the base of the PR and between 66920c7 and 09e7ac0.

📒 Files selected for processing (1)

• xtest/sdk/java/cli.sh

---

📝 Walkthrough

Walkthrough

The Java CLI script’s supports dpop probe now directly invokes the CLI’s DPoP support command instead of searching encryption help output for a flag.

Changes

DPoP feature probing

Layer / File(s)	Summary
Direct DPoP support check xtest/sdk/java/cli.sh	The dpop feature probe now runs java -jar ... supports dpop directly, removing the previous help-output grep and pipefail guard.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

• opentdf/tests#538: Updates related feature-probing logic in xtest/sdk/java/cli.sh.

Suggested reviewers: sujankota

Poem

I’m a bunny with a probe to run,
DPoP support checked direct in the sun.
No grep through help, no flags to seek,
One simple command makes the check sleek.
Hop, hop—cleaner logic this week!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly summarizes the main change: switching Java DPoP capability detection to the supports subcommand.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch java-dpop-supports-detection

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

X-Test Results

✅ java@main-v0.20.0
✅ go@v0.35.0-main
✅ java@v0.17.1-main
✅ java@main-main
✅ js@main-v0.20.0
✅ js@main-main
✅ go@main-v0.20.0
✅ go@main-main
✅ java@v0.17.1-v0.20.0
✅ js@v0.4.0-main
✅ go@v0.35.0-v0.20.0
✅ js@v0.4.0-v0.20.0