Skip to content

Conversation

@jmetrikat
Copy link
Collaborator

No description provided.

Copy link

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 adds support for customizing the API listen address via a --listen-address flag for both the anytype serve and anytype service install commands. It also refactors the service command structure by extracting each service subcommand (install, uninstall, start, stop, restart, status) into separate packages for better code organization.

  • Adds --listen-address flag to serve and service install commands to allow customization of the JSON API endpoint
  • Refactors service subcommands into individual packages for improved modularity
  • Updates serviceprogram.Program to track and use the custom API listen address for authentication

Reviewed changes

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

Show a summary per file
File Description
core/serviceprogram/serviceprogram.go Adds GetService and GetServiceWithAddress functions, updates Program struct to store API listen address, and uses it during auto-login
cmd/serve/serve.go Adds --listen-address flag and passes it to the Program constructor
cmd/service/install/install.go New file for install subcommand with --listen-address flag support
cmd/service/uninstall/uninstall.go New file for uninstall subcommand
cmd/service/start/start.go New file for start subcommand
cmd/service/stop/stop.go New file for stop subcommand
cmd/service/restart/restart.go New file for restart subcommand
cmd/service/status/status.go New file for status subcommand
cmd/service/service.go Refactored to import and register subcommands from separate packages

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

Copy link

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 9 out of 9 changed files in this pull request and generated 5 comments.


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

@jmetrikat jmetrikat merged commit 696dbef into main Dec 16, 2025
2 checks passed
@jmetrikat jmetrikat deleted the fix/add-missing-listen-addr-flag branch December 17, 2025 00:28
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