chore: studio should only us appid parameter for queries

[martinothamar]

Description

If I understood correctly, using both of these as predicates leads to worse perf, Org is redundant when we already have AppId

Related Issue(s)

• #{issue number}

Verification

• Your code builds clean without any errors or warnings
• Manual testing done (required)
• Relevant automated test added (if you find this hard, leave it and we'll help out)
• All tests run green

Documentation

• User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

Summary by CodeRabbit

• Bug Fixes
  • Fixed handling of studio instance parameters so an incorrect organization value is no longer applied.
  • Preserved the combined app identifier format to ensure existing routing and lookups continue to work.
  • Prevents potential misrouting or parameter conflicts caused by the previous assignment.

[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: fc8f21dd-ca7f-4930-b845-00304d261f3d

📥 Commits

Reviewing files that changed from the base of the PR and between b3b9595 and e1897b5.

📒 Files selected for processing (1)

• src/Storage/Models/StudioInstanceParameters.cs

💤 Files with no reviewable changes (1)

• src/Storage/Models/StudioInstanceParameters.cs

---

📝 Walkthrough

Walkthrough

The StudioInstanceParameters.ToInstanceQueryParameters() object initializer no longer assigns the Org field to the returned InstanceQueryParameters; AppId remains set as "{Org}/{App}" and all other mappings are unchanged.

Changes

Instance Query Parameter Mapping

Layer / File(s)	Summary
Remove Org assignment src/Storage/Models/StudioInstanceParameters.cs	Omit Org from the InstanceQueryParameters object initializer in ToInstanceQueryParameters(); AppId continues to be set to "{Org}/{App}" and other properties are preserved.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 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 contains a typo ('us' instead of 'use') and is somewhat vague, but it clearly indicates the main change: removing the Org parameter from queries.
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.

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

✨ 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 chore/studio-instance-parameters-only-appid

---

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

[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