[NO-REF] Update Claude PR Review workflow to remove deprecated arguments

[pedro-lb]

PR Description

This PR adds the following changes:

• Removes deprecated arguments from the Claude PR review workflow
  • hide_previous_comments
  • inline_comments

💡 note: We recently switched to a new version of the "Claude PR Review" workflow that does inline comments by default, in order to improve the code review process. Please refer to this PR for more context: https://github.com/Constructor-io/shared-claude-code-resources/pull/39

🕹 Demo

Once this is merged you should see that the "Claude PR Review" workflow no longer failing, and giving you inline reviews.

You'll also see a short summary in the code review body, with all discussions extracted as inline comments in the code.

Example:

[constructor-claude-bedrock]

Code Review Summary

This is a minimal, focused maintenance PR that removes two deprecated arguments (hide_previous_comments and inline_comments) from the Claude PR Review GitHub Actions workflow (.github/workflows/claude-review.yml). The change is straightforward: the with: block and the hide_previous_comments: true entry are removed entirely, as the new version of the shared workflow handles inline comments natively and no longer supports these parameters.

Detailed Feedback

• [File: .github/workflows/claude-review.yml] The removal is correct and clean. The diff is exactly what is needed — no leftover empty with: block, no extraneous whitespace changes. The resulting workflow correctly delegates all configuration to the reusable workflow via secrets: inherit, which is the intended pattern.

• [General] The PR description is clear and includes a helpful reference to the upstream shared-resources PR for context. No functional Java code is changed, so there is no risk of regression in the library itself.

• [General] One minor observation: the PR description mentions removing both hide_previous_comments and inline_comments as deprecated arguments. However, the actual diff only shows the removal of the with: block containing hide_previous_comments: true. There is no evidence of a separate inline_comments argument having existed in this file. This is likely a slight inaccuracy in the description (the inline_comments argument may have been removed previously or may not have existed in this repo's copy of the workflow), but worth confirming that both deprecated arguments have been fully handled as intended.

Conclusion

The change is correct, minimal, and low-risk. The workflow file is left in a clean and valid state. Approved pending the minor description clarification above.

[Copilot]

Pull request overview

Updates the repository’s “Claude PR Review” GitHub Actions workflow to align with the newer shared workflow interface by removing deprecated inputs, preventing workflow failures and ensuring inline reviews work as intended.

Changes:

• Removed the deprecated hide_previous_comments workflow input from the reusable workflow invocation.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.