fix: linter issues - octal literals and alignment
This commit is contained in:
parent
3646d92a5a
commit
8fb9c6d813
3 changed files with 10 additions and 10 deletions
|
|
@ -102,7 +102,7 @@ func listPlugins() ([]string, error) {
|
||||||
}
|
}
|
||||||
|
|
||||||
// Check if executable by anyone
|
// Check if executable by anyone
|
||||||
if info.Mode()&0111 != 0 {
|
if info.Mode()&0o111 != 0 {
|
||||||
plugins = append(plugins, entry.Name())
|
plugins = append(plugins, entry.Name())
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
@ -120,7 +120,7 @@ func findPlugin(name string) (string, error) {
|
||||||
|
|
||||||
// Try exact match first
|
// Try exact match first
|
||||||
pluginPath := filepath.Join(pluginsDir, name)
|
pluginPath := filepath.Join(pluginsDir, name)
|
||||||
if info, err := os.Stat(pluginPath); err == nil && !info.IsDir() && info.Mode()&0111 != 0 {
|
if info, err := os.Stat(pluginPath); err == nil && !info.IsDir() && info.Mode()&0o111 != 0 {
|
||||||
return pluginPath, nil
|
return pluginPath, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
@ -128,7 +128,7 @@ func findPlugin(name string) (string, error) {
|
||||||
extensions := []string{".sh", ".bash", ".py", ".go", ""}
|
extensions := []string{".sh", ".bash", ".py", ".go", ""}
|
||||||
for _, ext := range extensions {
|
for _, ext := range extensions {
|
||||||
extPath := pluginPath + ext
|
extPath := pluginPath + ext
|
||||||
if info, err := os.Stat(extPath); err == nil && !info.IsDir() && info.Mode()&0111 != 0 {
|
if info, err := os.Stat(extPath); err == nil && !info.IsDir() && info.Mode()&0o111 != 0 {
|
||||||
return extPath, nil
|
return extPath, nil
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
@ -145,7 +145,7 @@ func findPlugin(name string) (string, error) {
|
||||||
if err != nil {
|
if err != nil {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
if info.Mode()&0111 == 0 {
|
if info.Mode()&0o111 == 0 {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
pluginName := entry.Name()
|
pluginName := entry.Name()
|
||||||
|
|
|
||||||
|
|
@ -75,7 +75,7 @@ func TestRunPluginNotFound(t *testing.T) {
|
||||||
func TestListPluginsIntegration(t *testing.T) {
|
func TestListPluginsIntegration(t *testing.T) {
|
||||||
// This is an integration test that checks the actual plugins directory
|
// This is an integration test that checks the actual plugins directory
|
||||||
pluginsDir := getPluginsDir()
|
pluginsDir := getPluginsDir()
|
||||||
|
|
||||||
// Create a temporary test plugin
|
// Create a temporary test plugin
|
||||||
tmpDir := pluginsDir
|
tmpDir := pluginsDir
|
||||||
if err := os.MkdirAll(tmpDir, 0755); err != nil {
|
if err := os.MkdirAll(tmpDir, 0755); err != nil {
|
||||||
|
|
|
||||||
|
|
@ -29,11 +29,11 @@ func NewPicoclawCommand() *cobra.Command {
|
||||||
short := fmt.Sprintf("%s picoclaw - Personal AI Assistant v%s\n\n", internal.Logo, internal.GetVersion())
|
short := fmt.Sprintf("%s picoclaw - Personal AI Assistant v%s\n\n", internal.Logo, internal.GetVersion())
|
||||||
|
|
||||||
cmd := &cobra.Command{
|
cmd := &cobra.Command{
|
||||||
Use: "picoclaw",
|
Use: "picoclaw",
|
||||||
Short: short,
|
Short: short,
|
||||||
Example: "picoclaw list",
|
Example: "picoclaw list",
|
||||||
Args: rootArgsValidator,
|
Args: rootArgsValidator,
|
||||||
ValidArgsFunction: rootCompleteArgs,
|
ValidArgsFunction: rootCompleteArgs,
|
||||||
DisableFlagParsing: true, // Pass all args (including flags) directly to plugins/commands
|
DisableFlagParsing: true, // Pass all args (including flags) directly to plugins/commands
|
||||||
RunE: func(cmd *cobra.Command, args []string) error {
|
RunE: func(cmd *cobra.Command, args []string) error {
|
||||||
// With DisableFlagParsing, --help is just an arg - show help
|
// With DisableFlagParsing, --help is just an arg - show help
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue