Support for anonymous_profile_tracker parameter

[SujanSanjula96]

Proposed changes in this pull request

Support for anonymous_profile_tracker parameter during the registration flow.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 221a99fb-b249-43e4-b23b-b1938bdf0ca4

📥 Commits

Reviewing files that changed from the base of the PR and between c97c4ea and b8bd25b.

📒 Files selected for processing (2)

• components/flow-orchestration-framework/org.wso2.carbon.identity.flow.execution.engine/src/main/java/org/wso2/carbon/identity/flow/execution/engine/Constants.java
• components/flow-orchestration-framework/org.wso2.carbon.identity.flow.execution.engine/src/main/java/org/wso2/carbon/identity/flow/execution/engine/FlowExecutionService.java

---

📝 Walkthrough

Walkthrough

A new ANONYMOUS_PROFILE_TRACKER constant ("anonymous_profile_tracker") is added to Constants.java. FlowExecutionService.executeFlow is updated to extract an isInitiation boolean and, on the first request of the registration flow, move the tracker value from user input data into flow context properties.

Changes

Anonymous Profile Tracker Migration

Layer / File(s)	Summary
ANONYMOUS_PROFILE_TRACKER constant ...flow.execution.engine/Constants.java	Adds the ANONYMOUS_PROFILE_TRACKER public string constant with value "anonymous_profile_tracker" under user attribute constants.
Tracker migration in FlowExecutionService ...flow.execution.engine/FlowExecutionService.java	Adds a static import for the new constant, refactors the initiation check into an isInitiation boolean, and on the initial registration flow request transfers anonymous_profile_tracker from getUserInputData() into context properties, removing it from user inputs.

Possibly related issues

• Support anonymous_profile_tracker query parameter support for IS legacy registration flow identity-customer-data-service#285: This PR directly implements the anonymous_profile_tracker query parameter support referenced in that issue by adding the constant and migrating the value from user inputs into flow context properties during the initial registration request.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The pull request description is severely incomplete, containing only a brief one-sentence statement without addressing any of the required template sections.	Fill out all required sections from the template including Purpose, Goals, Approach, User stories, Release note, Documentation, and others; minimum viable description should include context, rationale, and implementation details.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Support for anonymous_profile_tracker parameter' clearly and concisely describes the main change: adding support for a new parameter in the system.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
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.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.}

[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

[codecov]

Codecov Report

❌ Patch coverage is 33.33333% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 52.65%. Comparing base (ad99960) to head (b8bd25b).
⚠️ Report is 54 commits behind head on master.

Files with missing lines	Patch %	Lines
...ty/flow/execution/engine/FlowExecutionService.java	33.33%	2 Missing and 2 partials ⚠️

❌ Your patch check has failed because the patch coverage (33.33%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #8179      +/-   ##
============================================
- Coverage     52.91%   52.65%   -0.27%     
- Complexity    20911    21125     +214     
============================================
  Files          2186     2197      +11     
  Lines        128869   130620    +1751     
  Branches      19209    19564     +355     
============================================
+ Hits          68189    68775     +586     
- Misses        52377    53449    +1072     
- Partials       8303     8396      +93     

Flag	Coverage Δ
unit	38.22% <33.33%> (+0.09%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.