Skip to content

Remove auto-clone and git identity (now in Scorpio) - #162

Merged
Predixx merged 1 commit into
devfrom
strip-scorpio-overlap
Jun 1, 2026
Merged

Remove auto-clone and git identity (now in Scorpio)#162
Predixx merged 1 commit into
devfrom
strip-scorpio-overlap

Conversation

@Predixx

@Predixx Predixx commented May 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Remove autoCloneIfNeeded() and configureGitIdentityFromEnv() from the Theia layer. The companion Scorpio extension now handles repository cloning and git identity setup in EduIDE environments.
  • Drop gitUri/gitUser/gitMail from TheiaEnvironment and remove GIT_URI/GIT_USER/GIT_MAIL from THEIA_ENV_VARS. The DataBridge poll now resolves as soon as ARTEMIS_URL + ARTEMIS_TOKEN arrive (faster startup).
  • Keep cloneRepositoryProgrammatic for user-initiated clone from exercise detail view.
  • Keep GIT_* in KNOWN_BRIDGE_KEYS so the diagnostic probe still reports full bridge state.

Companion PR: EduIDE/scorpio#183

Test plan

  • tsc --noEmit passes (0 errors)
  • eslint passes (0 errors)
  • vitest run passes (873/873 tests)
  • No dangling references to removed symbols (gitUri, gitUser, gitMail, autoCloneIfNeeded, configureGitIdentityFromEnv)
  • Manual clone from exercise detail still works (cloneRepositoryProgrammatic preserved)
  • Diagnostic command (artemis.showTheiaEnvironment) still probes GIT_* via bridge
  • Verify in EduIDE that Scorpio handles auto-clone while this extension handles auth

Strip Theia auto-clone and git identity configuration from the
extension. The companion Scorpio extension now handles repository
cloning and git user.name/user.email setup in EduIDE environments.

- Remove autoCloneIfNeeded() and configureGitIdentityFromEnv()
- Remove gitUri/gitUser/gitMail from TheiaEnvironment type
- Drop GIT_URI/GIT_USER/GIT_MAIL from THEIA_ENV_VARS (faster bridge poll)
- Keep cloneRepositoryProgrammatic for user-initiated clone
- Keep GIT_* in KNOWN_BRIDGE_KEYS for diagnostic probe
- Update error messages and comments
@Predixx
Predixx force-pushed the strip-scorpio-overlap branch from 3485e00 to 81077c8 Compare June 1, 2026 11:19
@Predixx
Predixx merged commit 8a70458 into dev Jun 1, 2026
10 checks passed
@Predixx
Predixx deleted the strip-scorpio-overlap branch June 1, 2026 11:30
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.

1 participant