Skip to content

Relocate Breakout to Internal-Games repository#27

Draft
Copilot wants to merge 2 commits intoSave-Reworkfrom
copilot/relocate-breakout-files
Draft

Relocate Breakout to Internal-Games repository#27
Copilot wants to merge 2 commits intoSave-Reworkfrom
copilot/relocate-breakout-files

Conversation

Copy link

Copilot AI commented Nov 27, 2025

Moves Breakout game files to Internal-Games repository and updates wigdosXP to load it externally.

Changes

  • Removed local game files:

    • apps/games/breakout.html
    • scripts/apps/applications/games/simple/breakout.js
  • Updated application path in applications.js:

breakout: new Application(
    ["breakout", "Breakout"],
    "other",
    false,
    ["external", "https://wigdos-inc.github.io/Internal-Games/OFES/breakout/breakout.html"]
),
  • Preserved: Icon file and SU task definitions (still reference by app ID)
Original prompt

This section details on the original issue you should resolve

<issue_title>Relocate Breakout</issue_title>
<issue_description>WigdosXP -> Internal-Games</issue_description>

<agent_instructions>The files should be removed from the wigdosXP branch and put into the Internal-Games repository</agent_instructions>

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


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: stu102871 <181106524+stu102871@users.noreply.github.com>
Copilot AI changed the title [WIP] Move Breakout files to Internal-Games repository Relocate Breakout to Internal-Games repository Nov 27, 2025
Copilot AI requested a review from stu102871 November 27, 2025 13:34
@stu102871
Copy link
Collaborator

@copilot Breakout is currently broken in Internal-Games because not all the references are correct. It's getting some error 404s

Copy link
Author

Copilot AI commented Nov 27, 2025

@copilot Breakout is currently broken in Internal-Games because not all the references are correct. It's getting some error 404s

The 404 errors are caused by incorrect file references in the Internal-Games repository. The OFES/breakout/breakout.html file has paths that still point to the old wigdosXP structure:

Issues in Internal-Games that need fixing:

  1. <base href="../../"> should be removed
  2. scripts/libraries/p5/p5.min.js../../globalScripts/libraries/p5/p5.min.js
  3. scripts/apps/games/simple/breakout.jsbreakout.js
  4. Background image and font references need local assets folder (like sublimator has)

Compare with the working OFES/sublimator/sublimator.html to see the correct structure. These changes need to be made in the Internal-Games repository, which I cannot modify from this PR.

@stu102871
Copy link
Collaborator

@copilot Can you create an issue tied to that repository and assign Copilot to that so it can be fixed in another session?

Copy link
Author

Copilot AI commented Nov 27, 2025

@copilot Can you create an issue tied to that repository and assign Copilot to that so it can be fixed in another session?

I'm unable to create issues in other repositories due to environment limitations. You'll need to manually create an issue in the Internal-Games repository with the following details:

Title: Fix Breakout file references for Internal-Games structure

Description:
The breakout.html has incorrect paths that cause 404 errors. Update OFES/breakout/breakout.html:

  1. Remove <base href="../../">
  2. Change scripts/libraries/p5/p5.min.js../../globalScripts/libraries/p5/p5.min.js
  3. Change scripts/libraries/p5/p5.sound.min.js../../globalScripts/libraries/p5/p5.sound.min.js
  4. Change scripts/apps/games/simple/breakout.jsbreakout.js
  5. Add local assets folder with fonts and background image (similar to sublimator)
  6. Update breakout.js to use local font path

Reference the working OFES/sublimator/sublimator.html for the correct structure.

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.

2 participants