Skip to content

Conversation

@giordano-lucas
Copy link
Member

@giordano-lucas giordano-lucas commented Jan 30, 2026

Greptile Overview

Greptile Summary

This PR updates the module path from github.com/salmonumbrella/notte-cli to github.com/nottelabs/notte-cli across the codebase, reflecting the repository's move to the nottelabs organization.

Key Changes:

  • Updated go.mod module declaration
  • Updated all import statements across 41 Go files
  • Updated .golangci.yml local-prefixes configuration
  • Updated GitHub links in HTML templates

Critical Issue:

  • internal/cmd/output_helpers.go was not included in this PR and still contains the old import path on line 8, which will cause compilation failures

Confidence Score: 1/5

  • This PR cannot be merged as-is due to a critical compilation error
  • The PR misses updating internal/cmd/output_helpers.go, which still imports github.com/salmonumbrella/notte-cli/internal/api. This will cause the code to fail compilation since the module path has changed
  • internal/cmd/output_helpers.go must be updated before merging

Important Files Changed

Filename Overview
go.mod Updated module path from salmonumbrella to nottelabs
cmd/notte/main.go Updated import path to use new module name
.golangci.yml Updated local-prefixes to match new module path
internal/cmd/output_helpers.go Not updated in this PR - still contains old import path causing compilation failure

@giordano-lucas giordano-lucas merged commit 8a407fa into main Jan 30, 2026
2 checks passed
@giordano-lucas giordano-lucas deleted the fix/install-path branch January 30, 2026 08:20
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

4 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

"reflect"

"github.com/salmonumbrella/notte-cli/internal/api"
"github.com/nottelabs/notte-cli/internal/api"
Copy link

Choose a reason for hiding this comment

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

This import still uses the old module path github.com/salmonumbrella/notte-cli and needs to be updated to github.com/nottelabs/notte-cli

Suggested change
"github.com/nottelabs/notte-cli/internal/api"
"github.com/nottelabs/notte-cli/internal/api"
Prompt To Fix With AI
This is a comment left during a code review.
Path: internal/cmd/output_helpers.go
Line: 8:8

Comment:
This import still uses the old module path `github.com/salmonumbrella/notte-cli` and needs to be updated to `github.com/nottelabs/notte-cli`

```suggestion
	"github.com/nottelabs/notte-cli/internal/api"
```

How can I resolve this? If you propose a fix, please make it concise.

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