Skip to content

feat: add filter scope annotations (@Global, @Route) - #67

Merged
VonAdamo merged 2 commits into
mainfrom
feat/filter-scope-annotations
Feb 17, 2026
Merged

feat: add filter scope annotations (@Global, @Route)#67
VonAdamo merged 2 commits into
mainfrom
feat/filter-scope-annotations

Conversation

@kristinaxm

@kristinaxm kristinaxm commented Feb 15, 2026

Copy link
Copy Markdown

Issue: #66

Summary by CodeRabbit

  • New Features
    • Added two new type-level annotations for runtime configuration (including ordering)
    • Support for adjustable execution order on annotated types
    • Support for annotating types with one or more routing paths for routing metadata

@kristinaxm kristinaxm self-assigned this Feb 15, 2026
@coderabbitai

coderabbitai Bot commented Feb 15, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

Adds two new public Java annotations in package org.juv25d.filter.annotation: @Global and @Route. Both target types, are retained at runtime, and expose an order() member (default 0); @Route also exposes a String[] value() element for paths.

Changes

Cohort / File(s) Summary
New annotations
src/main/java/org/juv25d/filter/annotation/Global.java, src/main/java/org/juv25d/filter/annotation/Route.java
Introduced public @interface Global with int order() default 0 and public @interface Route with String[] value() and int order() default 0. Both use @Target(ElementType.TYPE) and @Retention(RetentionPolicy.RUNTIME).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related issues

Suggested reviewers

  • addee1
  • mattknatt
  • annikaholmqvist94

Poem

🐇 I hopped through code to leave a mark,
Two little annotations bright in the dark.
Order set to zero, routes hold their way,
Type-level signals for runtime to play. ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 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: adding two new filter scope annotations (@Global and @Route), which matches the primary purpose of the pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/filter-scope-annotations

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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 and usage tips.

bamsemats
bamsemats previously approved these changes Feb 16, 2026
Comment thread src/main/java/org/juv25d/filter/annotation/Route.java
@VonAdamo
VonAdamo merged commit af32a5d into main Feb 17, 2026
2 checks passed
@kristinaxm
kristinaxm deleted the feat/filter-scope-annotations branch February 18, 2026 19:37
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.

3 participants