Skip to content

fix(opencode): align setup messaging with auto-start behavior#333

Open
egdev6 wants to merge 11 commits into
Gentleman-Programming:mainfrom
egdev6:fix/opencode-setup-auto-start-message
Open

fix(opencode): align setup messaging with auto-start behavior#333
egdev6 wants to merge 11 commits into
Gentleman-Programming:mainfrom
egdev6:fix/opencode-setup-auto-start-message

Conversation

@egdev6
Copy link
Copy Markdown
Contributor

@egdev6 egdev6 commented May 5, 2026

🔗 Linked Issue

Closes #300

🏷️ PR Type

  • type:bug — Bug fix
  • type:feature — New feature
  • type:docs — Documentation only
  • type:refactor — Code refactoring (no behavior change)
  • type:chore — Maintenance, dependencies, tooling
  • type:breaking-change — Breaking change

📝 Summary

  • Align OpenCode setup messaging with the actual plugin behavior by removing engram serve as a normal next step.
  • Update the CLI post-install output to explain that session tracking auto-starts the HTTP server when needed, with manual engram serve kept only as a fallback for restricted environments.
  • Sync the related OpenCode docs so the setup flow, plugin behavior, and troubleshooting guidance all tell the same story.

📂 Changes

File Change
cmd/engram/main.go Updated OpenCode post-install output to describe HTTP server auto-start and manual fallback messaging
cmd/engram/main_test.go Adjusted output expectations for the OpenCode post-install flow
docs/AGENT-SETUP.md Clarified that manual engram serve is fallback-only for restricted environments
docs/PLUGINS.md Aligned plugin documentation with normal auto-start behavior and fallback guidance

🧪 Test Plan

  • Unit tests pass locally: go test ./...

  • E2E tests pass locally: go test -tags e2e ./internal/server/...

  • Manually tested the affected functionality

  • Verified the updated post-install output via go test ./cmd/engram -run TestPrintPostInstall

  • Confirmed the wording matches the current plugin behavior and setup docs

🤖 Automated Checks

These run automatically and all must pass before merge:

Check What it verifies Status
Check Issue Reference PR body contains Closes #N / Fixes #N / Resolves #N ⏳
Check Issue Has status:approved Linked issue has status:approved label ⏳
Check PR Has type: Label* PR has exactly one type:* label ⏳
Unit Tests go test ./... passes ⏳
E2E Tests go test -tags e2e ./internal/server/... passes ⏳

✅ Contributor Checklist

  • I linked an approved issue above (Closes #N)
  • I added exactly one type:* label to this PR
  • I ran unit tests locally: go test ./...
  • I ran e2e tests locally: go test -tags e2e ./internal/server/...
  • Docs updated (if behavior changed)
  • Commits follow conventional commits format
  • No Co-Authored-By trailers in commits

💬 Notes for Reviewers

Copilot AI review requested due to automatic review settings May 5, 2026 13:10
Copilot AI reviewed May 5, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the OpenCode setup messaging so the CLI and docs better reflect the plugin’s existing behavior: the HTTP server is auto-started when needed, and manual engram serve is only meant as a fallback. It fits into the setup/documentation flow for agent integrations, especially engram setup opencode.

Changes:

  • Updated engram setup opencode post-install output to describe auto-start behavior.
  • Adjusted unit-test expectations for the new post-install text.
  • Synced the OpenCode setup/docs wording around fallback-only manual server startup.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
cmd/engram/main.go Revises OpenCode post-install next-step messaging.
cmd/engram/main_test.go Updates assertions for the revised CLI output.
docs/AGENT-SETUP.md Clarifies normal setup vs manual fallback in the setup guide.
docs/PLUGINS.md Aligns plugin docs with auto-start and fallback wording.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cmd/engram/main_test.go Outdated
Comment thread cmd/engram/main.go Outdated
@egdev6 egdev6 requested a review from Copilot May 5, 2026 13:20
Copilot AI reviewed May 5, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cmd/engram/main_test.go
Comment thread cmd/engram/main.go Outdated
Comment thread docs/AGENT-SETUP.md Outdated
@egdev6 egdev6 requested a review from Copilot May 5, 2026 13:29
Copilot AI reviewed May 5, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cmd/engram/main_test.go
@egdev6
Copy link
Copy Markdown
Contributor Author

egdev6 commented May 5, 2026

Ready for Review!

Copilot AI review requested due to automatic review settings May 7, 2026 06:59
Copilot AI reviewed May 7, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Comment thread cmd/engram/main_test.go Outdated
@egdev6
Copy link
Copy Markdown
Contributor Author

egdev6 commented May 7, 2026

@copilot apply changes based on the comments in this thread

@egdev6 egdev6 requested a review from Copilot May 7, 2026 08:00
Copilot AI reviewed May 7, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Comment thread cmd/engram/main_test.go
@egdev6
Copy link
Copy Markdown
Contributor Author

egdev6 commented May 7, 2026

@copilot apply changes based on the comments in this thread

@egdev6 egdev6 requested a review from Copilot May 7, 2026 08:06
Copilot AI reviewed May 7, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Comment thread cmd/engram/main_test.go
@egdev6 egdev6 requested a review from Copilot May 7, 2026 08:14
Copilot AI reviewed May 7, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings May 11, 2026 08:21
Copilot AI reviewed May 11, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Comment thread cmd/engram/main.go Outdated
@egdev6
Potential fix for pull request finding
6b848dc 
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 11, 2026 08:24
Copilot AI reviewed May 11, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings May 11, 2026 12:06
Copilot AI reviewed May 11, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

OpenCode setup still instructs manual engram serve even though the plugin auto-starts the HTTP server

2 participants

@egdev6