Skip to content

Commit

Permalink
Cleanup: Remove the ai-logs flag.
Browse files Browse the repository at this point in the history
* This flag is deprecated and no longer used.
* The frontend should have stopped using this flag starting in 3.7.5
* The frontend is now up to 3.8.5
* So we should be good to remove this flag because the frontend should no longer be setting it.

* Related to jlewi/foyle#211
  • Loading branch information
jlewi committed Oct 2, 2024
1 parent dc8d765 commit a1899d0
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions internal/cmd/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -142,11 +142,8 @@ The kernel is used to run long running processes like shells and interacting wit
cmd.Flags().StringVarP(&addr, "address", "a", defaultAddr, "Address to create unix (unix:///path/to/socket) or IP socket (localhost:7890)")
cmd.Flags().BoolVar(&devMode, "dev", false, "Enable development mode")
cmd.Flags().BoolVar(&enableRunner, "runner", true, "Enable runner service (legacy, defaults to true)")
// The AIFlag is no longer used, we can remove it as soon as the option has been removed from the frontend.
cmd.Flags().BoolVar(&enableAILogs, "ai-logs", false, "Enable logs to support training an AI")
cmd.Flags().StringVar(&tlsDir, "tls", defaultTLSDir, "Directory in which to generate TLS certificates & use for all incoming and outgoing messages")
cmd.Flags().StringVar(&configDir, configDirF, GetUserConfigHome(), "Sets the configuration directory.")
_ = cmd.Flags().MarkHidden("runner")
_ = cmd.Flags().MarkDeprecated("ai-logs", "This flag is no longer used.")
return &cmd
}

0 comments on commit a1899d0

Please sign in to comment.