Skip to content

feat: restore headless cli on shared service#194

Merged
ksamaschke merged 1 commit intodevfrom
codex/issue-191-cli-shared-service-20260314075856
Mar 14, 2026
Merged

feat: restore headless cli on shared service#194
ksamaschke merged 1 commit intodevfrom
codex/issue-191-cli-shared-service-20260314075856

Conversation

@ksamaschke
Copy link
Copy Markdown
Contributor

Summary

  • route headless CLI diagnostics and catalog access through the shared application service
  • reframe CLI help around headless automation while demoting browser-era serve/launch paths
  • add focused CLI shared-service contract coverage and stabilize the API readiness regression test

Testing

  • npm run clean:compiled-tests && ./node_modules/.bin/tsc -p tsconfig.json && node --test dist/tests/installer/cli-shared-service-red-phase.test.js dist/tests/installer/bootstrap-launch.test.js dist/tests/installer/cli-serve.test.js dist/tests/installer/cli-skills-publish.test.js dist/tests/installer/installations.test.js
  • PATH="/Users/karsten/Work/Development/intelligentcode-ai/intelligent-code-agents/node_modules/.bin:$PATH" npm test

Tracking

@ksamaschke
Copy link
Copy Markdown
Contributor Author

ICA-REVIEW
ICA-REVIEW-RECEIPT
Reviewer-Stage: 3 (temp checkout)
Reviewer-Agent: reviewer (subagent)
PR: #194
Base: dev
Head-SHA: 3d55377
Date-UTC: 2026-03-14T07:28:09Z

Findings: 0
NO FINDINGS

Checks/Tests:

  • isolated temp-checkout diff review (PASS)
  • npm test (PASS, local branch validation before PR)
  • required GitHub checks: memory-protection (PASS)

Notes:

  • No additional required checks were configured on this PR beyond memory-protection.

Result: PASS

@ksamaschke ksamaschke force-pushed the codex/issue-191-cli-shared-service-20260314075856 branch from 3d55377 to ab0e0dc Compare March 14, 2026 07:34
@ksamaschke
Copy link
Copy Markdown
Contributor Author

ICA-REVIEW
ICA-REVIEW-RECEIPT
Reviewer-Stage: 3 (temp checkout)
Reviewer-Agent: reviewer (subagent)
PR: #194
Base: dev
Head-SHA: ab0e0dc
Date-UTC: 2026-03-14T07:35:21Z

Findings: 0
NO FINDINGS

Checks/Tests:

  • isolated temp-checkout diff review (PASS)
  • focused post-rebase verification: cli-shared-service-red-phase, cli-serve, api-security (PASS)
  • GitHub checks: test (PASS), memory-protection (PASS)

Notes:

  • Rebase kept the newer desktop-first CLI removal messaging from dev and preserved the shared-service contract changes from this PR.

Result: PASS

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