Skip to content

feat(go2.lic): preserve auxiliary scripts for one trip - #2458

Merged
mrhoribu merged 2 commits into
elanthia-online:masterfrom
therealatari:feat/go2-preserve-scripts
Sep 10, 2026
Merged

mrhoribu merged 2 commits into
elanthia-online:masterfrom
therealatari:feat/go2-preserve-scripts

Conversation

@therealatari

Copy link
Copy Markdown
Contributor

Summary

  • add a one-trip --preserve-scripts option to go2
  • leave roomnumbers and textsubs running for that trip instead of stopping and restarting them
  • keep the existing behavior as the default and avoid persisting the option

This narrow flag supports supervised callers that require uninterrupted observation provenance during bounded travel. It was split from #2456 at maintainer request so go2 history and blame remain independently scoped.

Verification

  • 5 focused examples, 0 failures
  • changed go2 script and specification pass RuboCop
  • both files pass Ruby syntax validation
  • the previously reviewed combined integration stack passed the full EO Scripts suite: 22,685 examples, 0 failures

@coderabbitai

coderabbitai Bot commented Sep 9, 2026 •

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: c23fc76b-5244-4d31-8efe-c20fdcf1a2cd

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@mrhoribu
mrhoribu merged commit e0dfc92 into elanthia-online:master Sep 10, 2026
4 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