Skip to content

feat(startup): Add structured launcher readiness protocol#1232

Merged
leumor merged 1 commit intodevelopfrom
feature/launcher-readiness-protocol
Apr 4, 2026
Merged

feat(startup): Add structured launcher readiness protocol#1232
leumor merged 1 commit intodevelopfrom
feature/launcher-readiness-protocol

Conversation

@leumor
Copy link
Copy Markdown
Collaborator

@leumor leumor commented Apr 4, 2026

Summary

Add a structured, versioned launcher-readiness handshake under the resolved runtime directory.

  • add shared readiness-file helpers in :foundation-fs
  • publish readiness from runtime only after the legacy HTTP shell is ready for normal use
  • make the desktop launcher prefer the readiness file, keep narrow legacy fallback behavior, and handle configured run-directory and daemon-log discovery
  • update focused tests and README wording for the new readiness contract
  • keep supporting warning-cleanup changes needed to leave the touched launcher/test code clean

How to test

  • ./gradlew compileTestJava
  • ./gradlew test --tests 'network.crypta.launcher.LauncherControllerTest'
  • ./gradlew test

Notes

  • Base branch: develop
  • Branch: feature/launcher-readiness-protocol

Add a versioned readiness file under the resolved runtime directory and publish it after the legacy HTTP shell finishes normal startup. Update the desktop launcher to prefer the structured readiness signal, keep narrow legacy fallback handling, and cover the new startup coordination paths in focused tests and README wording.
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 4, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
71.6% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@leumor leumor merged commit cc67662 into develop Apr 4, 2026
7 of 8 checks passed
@leumor leumor deleted the feature/launcher-readiness-protocol branch April 4, 2026 17: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