mirror of
https://github.com/sipeed/picoclaw.git
synced 2026-06-12 18:08:54 +00:00
Ruslan Semagin
73f27803d4
* refactor(cli): migrate to Cobra-based command structure Refactor CLI to use Cobra instead of manual os.Args parsing. - Introduce root command and structured subcommands under cmd/picoclaw/internal - Convert agent, auth, cron, gateway, migrate, onboard, skills, status and version to Cobra commands - Replace manual flag parsing with Cobra flags - Remove direct os.Args usage from command handlers - Keep existing command behavior and output semantics This change focuses on CLI structure and maintainability. No business logic changes intended. * chore(cli): remove version2 alias and make cobra a direct dependency * test(cli): add basic command tests - Add tests for CLI command tree and flag parsing - Align LDFLAGS injection path for version info - Remove unused manual help function * test: migrate command tests to testify assertions Replace standard library testing error checks (t.Error*, t.Fatalf) with assert/require from stretchr/testify across all cobra command tests for improved readability and consistency. * fix(cli): make linter happy * test: avoid duplication in windows config path test * test: simplify allowed command checks using slices.Contains * fix(skills): register subcommands during command construction - Move subcommand registration out of PersistentPreRunE - Ensure `picoclaw skills <subcommand>` resolves correctly - Minor install command and test cleanups * refactor(cli): address review feedback and improve command clarity * fix(authLogoutCmd): rm os.Exit
45 lines
1.1 KiB
Go
45 lines
1.1 KiB
Go
package cron
|
|
|
|
import (
|
|
"fmt"
|
|
"path/filepath"
|
|
|
|
"github.com/spf13/cobra"
|
|
|
|
"github.com/sipeed/picoclaw/cmd/picoclaw/internal"
|
|
)
|
|
|
|
func NewCronCommand() *cobra.Command {
|
|
var storePath string
|
|
|
|
cmd := &cobra.Command{
|
|
Use: "cron",
|
|
Aliases: []string{"c"},
|
|
Short: "Manage scheduled tasks",
|
|
Args: cobra.NoArgs,
|
|
RunE: func(cmd *cobra.Command, _ []string) error {
|
|
return cmd.Help()
|
|
},
|
|
// Resolve storePath at execution time so it reflects the current config
|
|
// and is shared across all subcommands.
|
|
PersistentPreRunE: func(_ *cobra.Command, _ []string) error {
|
|
cfg, err := internal.LoadConfig()
|
|
if err != nil {
|
|
return fmt.Errorf("error loading config: %w", err)
|
|
}
|
|
storePath = filepath.Join(cfg.WorkspacePath(), "cron", "jobs.json")
|
|
return nil
|
|
},
|
|
}
|
|
|
|
cmd.AddCommand(
|
|
newListCommand(func() string { return storePath }),
|
|
newAddCommand(func() string { return storePath }),
|
|
newRemoveCommand(func() string { return storePath }),
|
|
newEnableCommand(func() string { return storePath }),
|
|
newDisableCommand(func() string { return storePath }),
|
|
)
|
|
|
|
return cmd
|
|
}
|