Skip to content

fix: Keep __github_worker alive in battles - #1481

Merged
EttyKitty merged 6 commits into
Adeptus-Dominus:mainfrom
EttyKitty:fix/gh-worker
Aug 27, 2026
Merged

fix: Keep __github_worker alive in battles#1481
EttyKitty merged 6 commits into
Adeptus-Dominus:mainfrom
EttyKitty:fix/gh-worker

Conversation

@EttyKitty

@EttyKitty EttyKitty commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Closes #1479


Summary by cubic

Keeps the GitHub worker alive during battle transitions so instance_deactivate_all no longer crashes the IDE.

  • Introduces instance_deactivate_all_safe() to re-activate __github_worker and critical UI instances after battle-related deactivations.
  • Calls __GitHubEnsureInstance() before checking the worker's server in the OAuth, tick, and shutdown paths, and centralizes those checks in __GitHubHasActiveRequest().
  • Renames artifact_functions and instance_create to scr_artifact_functions and scr_instance_functions.
  • Makes IDE warnings non-fatal instead of aborting.

Written for commit 645cabf. Summary will update on new commits.

Review in cubic

@github-actions github-actions Bot added Size: Tiny Type: Fix This is a fix for a bug labels Aug 27, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 16 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread scripts/GitHubOAuth/GitHubOAuth.gml Outdated
Comment thread scripts/__GitHubRequestServerShutdown/__GitHubRequestServerShutdown.gml Outdated
Comment thread scripts/__GitHubWarn/__GitHubWarn.gml Outdated
Comment thread scripts/GitHubOAuth/GitHubOAuth.gml Outdated
Comment thread scripts/__GitHubTick/__GitHubTick.gml Outdated
Comment thread scripts/__GitHubEnsureInstance/__GitHubEnsureInstance.gml Outdated
Comment thread objects/obj_fleet/Create_0.gml Outdated

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 4 files (changes from recent commits).

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread scripts/__GitHubRequestServerShutdown/__GitHubRequestServerShutdown.gml Outdated
Comment thread scripts/GitHubOAuth/GitHubOAuth.gml Outdated

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 21 files (changes from recent commits).

Tip: Review your code locally with the cubic CLI to iterate faster.

Re-trigger cubic

Comment thread scripts/GitHubOAuth/GitHubOAuth.gml Outdated
Comment thread scripts/scr_instance_functions/scr_instance_functions.gml
Comment thread scripts/GitHubOAuth/GitHubOAuth.gml Outdated
Comment thread scripts/__GitHubTick/__GitHubTick.gml
Comment thread scripts/GitHubOAuth/GitHubOAuth.gml
@EttyKitty
EttyKitty merged commit 4125a9a into Adeptus-Dominus:main Aug 27, 2026
3 checks passed
@EttyKitty
EttyKitty deleted the fix/gh-worker branch August 27, 2026 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Size: Small Type: Fix This is a fix for a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

__github_worker deactivation throwing fatal exception

1 participant