Skip to content

Release open source!#6

Merged
rdbtCVS merged 14 commits intomainfrom
open-sourcing-this-bitch
Apr 17, 2026
Merged

Release open source!#6
rdbtCVS merged 14 commits intomainfrom
open-sourcing-this-bitch

Conversation

@rdbtCVS
Copy link
Copy Markdown
Member

@rdbtCVS rdbtCVS commented Apr 16, 2026

@rdbtCVS rdbtCVS requested a review from EpsilonPhoenix April 16, 2026 23:59
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d85e103e18

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 563 to 564
isLoaded = true
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Restore startup execution of the V5 module entrypoint

onInitialize() now only validates the token and marks the loader as loaded, but it no longer executes the downloaded V5 module entrypoint. ModuleManager.setup() (called from CTMixinPlugin.onLoad) only prepares module metadata/jars and does not run entries (entryPass is only called from CTJS.load()), so after a normal game start the V5 scripts never run until the user manually reloads CT.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Comment on lines 293 to 294
shutDownHard()
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Check dev-mode short-circuit before requiring auth

onMixinPlugin() fetches and validates a JWT before checking isLocalDeveloperModeEnabled(). In local/offline development (or any run without a token), this throws before reaching the developer-mode branch, so the documented local module workflow is blocked even when modules/V5 already exists. The developer-mode bypass should happen before hard-failing on token absence.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

why would you ever develop a macro while offline? hello? darioush propaganda bot.

@EpsilonPhoenix
Copy link
Copy Markdown
Member

Grrrr.

Okay fine, I will allow you to open source it with pathfinder, I'm too tired to argue.

Just please spam README or smth with the GPL license repeatedly, so people know that their skidware has to be open too cough hexis cough

@rdbtCVS
Copy link
Copy Markdown
Member Author

rdbtCVS commented Apr 17, 2026

Grrrr.

Okay fine, I will allow you to open source it with pathfinder, I'm too tired to argue.

Just please spam README or smth with the GPL license repeatedly, so people know that their skidware has to be open too cough hexis cough

my thinking was that hopefully it would be easier for them to just like vibecode there own modules in v5 rather than skid an entire client.

@rdbtCVS rdbtCVS merged commit d92a451 into main Apr 17, 2026
6 of 7 checks passed
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