Skip to content

Prepare Openza Flow for Microsoft Store - #22

Merged
solankydev merged 2 commits into
mainfrom
feature/ms-store-release
Jul 21, 2026
Merged

Prepare Openza Flow for Microsoft Store#22
solankydev merged 2 commits into
mainfrom
feature/ms-store-release

Conversation

@solankydev

@solankydev solankydev commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • associate the MSIX package identity with the Microsoft Store product while keeping a separate development manifest
  • add correctly sized Store branding assets and release packaging configuration
  • clean up compiled XAML bindings and document Microsoft Store installation

Impact

Prepares Openza Flow 1.0.0 for Microsoft Store packaging and distribution without committing generated packages, certificates, tokens, or private Store artifacts.

Validation

  • dotnet restore Openza.Flow.slnx
  • dotnet test src/Openza.Flow.Tests/Openza.Flow.Tests.csproj -c Release --no-restore — 41 passed
  • Release and Debug app builds — 0 warnings, 0 errors
  • Windows App Certification Kit — PASS
  • gitleaks detect --source . --verbose — clean

Summary by CodeRabbit

  • Documentation

    • Updated release documentation to identify V1 as Windows-first.
    • Clarified Microsoft Store packaging identities and submission steps.
  • New Features

    • Added a separate development package configuration for Windows builds.
  • Bug Fixes

    • Improved package identity and signing configuration for Store submissions.
    • Optimized static interface bindings for more consistent display behavior.
  • Chores

    • Excluded generated Store association files from source control.

@coderabbitai

coderabbitai Bot commented Jul 19, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 779f667b-fc8f-4137-a340-eb0f910891a2

📥 Commits

Reviewing files that changed from the base of the PR and between c8f767e and 1f0e196.

📒 Files selected for processing (2)
  • .gitignore
  • src/Openza.Flow/Package.Dev.appxmanifest
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/Openza.Flow/Package.Dev.appxmanifest

📝 Walkthrough

Walkthrough

The changes configure separate Debug and release Windows Store manifests, update packaging settings and documentation, ignore generated Store metadata, and switch static pull request row bindings from one-way to one-time updates.

Changes

Windows Store packaging

Layer / File(s) Summary
Package identities and Store metadata
src/Openza.Flow/Package.Dev.appxmanifest, src/Openza.Flow/Package.appxmanifest, .gitignore
Adds the development package manifest, updates the release publisher certificate, and ignores generated Store association metadata.
Configuration-dependent packaging build
src/Openza.Flow/Openza.Flow.csproj, README.md
Enables self-contained packaging, adjusts package revision and symbol settings, selects manifests by configuration, and documents release/debug Store identities and signed submissions.

Pull request row bindings

Layer / File(s) Summary
Static row binding modes
src/Openza.Flow/Controls/PullRequestRowControl.xaml
Changes row layout, text, visibility, and badge padding bindings from OneWay to OneTime.

Estimated code review effort: 2 (Simple) | ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the main change: preparing Openza Flow for Microsoft Store packaging.
Description check ✅ Passed The description covers summary, impact, and validation, though it omits the template's Notes and Screenshots sections.
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 unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/ms-store-release

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/Openza.Flow/Package.Dev.appxmanifest`:
- Around line 23-34: Add the startup, toast, and COM extension declarations to
the manifest’s Application element alongside the existing uap:VisualElements
block, using the required desktop:Extension and com:Extension entries for
registration. Set the startup task extension’s Enabled attribute to false, and
preserve the existing application metadata and opt-in behavior.

In `@src/Openza.Flow/Package.StoreAssociation.xml`:
- Around line 369-374: Remove the tracked Package.StoreAssociation.xml artifact
containing Store-private account and Partner Center metadata, and add
Package.StoreAssociation.xml to the repository’s .gitignore so future Store
association wizard output is excluded. Do not replace it with a sanitized copy;
contributors should regenerate it locally when needed.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 6af1413e-2127-4fec-b13c-55db7abf5f2e

📥 Commits

Reviewing files that changed from the base of the PR and between f0e7db8 and c8f767e.

⛔ Files ignored due to path filters (6)
  • src/Openza.Flow/Assets/Square150x150Logo.png is excluded by !**/*.png
  • src/Openza.Flow/Assets/Square44x44Logo.png is excluded by !**/*.png
  • src/Openza.Flow/Assets/Square44x44Logo.targetsize-32_altform-unplated.png is excluded by !**/*.png
  • src/Openza.Flow/Assets/Square44x44Logo.targetsize-48.png is excluded by !**/*.png
  • src/Openza.Flow/Assets/StoreLogo.png is excluded by !**/*.png
  • src/Openza.Flow/Assets/Wide310x150Logo.png is excluded by !**/*.png
📒 Files selected for processing (6)
  • README.md
  • src/Openza.Flow/Controls/PullRequestRowControl.xaml
  • src/Openza.Flow/Openza.Flow.csproj
  • src/Openza.Flow/Package.Dev.appxmanifest
  • src/Openza.Flow/Package.StoreAssociation.xml
  • src/Openza.Flow/Package.appxmanifest

Comment thread src/Openza.Flow/Package.Dev.appxmanifest
Comment thread src/Openza.Flow/Package.StoreAssociation.xml Outdated
- Fix P1: register startup, toast, and COM extensions in the Debug manifest (#3611007986)
- Fix P1: remove and ignore Store association metadata (#3611007987)

Addresses: #3611007986, #3611007987
@solankydev
solankydev merged commit fd3dbce into main Jul 21, 2026
2 checks passed
@solankydev
solankydev deleted the feature/ms-store-release branch July 21, 2026 15:35
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.

1 participant