Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 30, 2025

Migrate the Android project from MonoGame to KNI (nkast.Xna.Framework), matching the existing Blazor project pattern.

Changes

Package migration:

  • Replace MonoGame.Framework.Androidnkast.Kni.Platform.Android.GL + nkast.Xna.Framework.* packages
  • Replace MonoGame.Content.Builder.Tasknkast.Xna.Framework.Content.Pipeline.Builder

Build configuration:

  • Add <KniPlatform>Android</KniPlatform> property
  • Rename MonoGameContentReferenceKniContentReference
  • Add BLAZOR define to exclude SixLabors.ImageSharp code (same pattern as Blazor project)

Project structure:

  • Remove ProjectReference to MonoGame.Core
  • Import shared project directly via DeveMazeGeneratorCore.MonoGame.Shared.projitems
  • Update solution file with shared project reference
<!-- Before -->
<PackageReference Include="MonoGame.Framework.Android" Version="3.8.4.1" />
<MonoGameContentReference Include="..\...\Content.mgcb" />
<ProjectReference Include="..\DeveMazeGeneratorCore.MonoGame.Core\..." />

<!-- After -->
<PackageReference Include="nkast.Kni.Platform.Android.GL" Version="4.2.9001.1" />
<PackageReference Include="nkast.Xna.Framework" Version="4.2.9001" />
<!-- ... other nkast.Xna.Framework.* packages -->
<KniContentReference Include="..\...\Content.mgcb" />
<Import Project="..\DeveMazeGeneratorCore.MonoGame.Shared\..." />

Note: Content builder (MGCB.exe) only runs on Windows, consistent with existing GitHub Actions workflow.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com//advisories
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>migrate Android project to use kni</issue_title>
<issue_description>Just like the other projects I want the android project to also use kni. See the migration guide and the Blazor project as examples. Search for all kni packages on nuget and make sure to do a complete build of everything just like in the GitHub actions</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

- Replace MonoGame.Framework.Android with KNI packages (nkast.Xna.Framework.*, nkast.Kni.Platform.Android.GL)
- Replace MonoGame.Content.Builder.Task with nkast.Xna.Framework.Content.Pipeline.Builder
- Change MonoGameContentReference to KniContentReference
- Add KniPlatform property for content building
- Import shared project directly instead of referencing MonoGame.Core (matches Blazor approach)
- Add BLAZOR define to exclude SixLabors.ImageSharp code (like Blazor project)
- Update solution file to include shared project reference for Android

Co-authored-by: devedse <2350015+devedse@users.noreply.github.com>
Copilot AI changed the title [WIP] Migrate Android project to use kni Migrate Android project to use KNI framework Nov 30, 2025
Copilot AI requested a review from devedse November 30, 2025 03:22
@codecov
Copy link

codecov bot commented Nov 30, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 53.54%. Comparing base (6ef3381) to head (53ee47b).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #104   +/-   ##
=======================================
  Coverage   53.54%   53.54%           
=======================================
  Files          52       52           
  Lines        1733     1733           
  Branches      385      385           
=======================================
  Hits          928      928           
  Misses        805      805           

☔ View full report in Codecov by Sentry.
📢 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.

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.

migrate Android project to use kni

2 participants