refactor(pr): pr comments

internal/jimm/jimm.go

@@ -500,7 +500,7 @@ type API interface {
 	// can list models for any user (at this stage).  Other users
 	// can only ask about their own models.
 	//
-	// In our wrapper, we ask for the owner. So expect ALL models from
+	// In our wrapper, we ask as the controller admin. So expect ALL models from
 	// the controller.
 	ListModels(ctx context.Context) ([]base.UserModel, error)
 }

internal/jimm/model.go

@@ -1358,13 +1358,13 @@ func (j *JIMM) ListModels(ctx context.Context, user *openfga.User) ([]base.UserM
 	const op = errors.Op("jimm.ListModels")
 	zapctx.Info(ctx, string(op))
 
-	// Get models uuids user has access to
+	// Get uuids of models the user has access to
 	uuids, err := user.ListModels(ctx, ofganames.ReaderRelation)
 	if err != nil {
-		return nil, errors.E(op, err, "failed to list models")
+		return nil, errors.E(op, err, "failed to list user models")
 	}
 
-	// Get the models themselves
+	// Get the models from the database
 	models, err := j.DB().GetModelsByUUID(ctx, uuids)
 	if err != nil {
 		return nil, errors.E(op, err, "failed to get models by uuid")

internal/jujuclient/modelmanager.go

@@ -333,7 +333,7 @@ func (c Connection) ChangeModelCredential(ctx context.Context, model names.Model
 // can list models for any user (at this stage).  Other users
 // can only ask about their own models.
 //
-// In our wrapper, we ask for the owner. So expect ALL models from
+// In our wrapper, we ask as the controller admin. So expect ALL models from
 // the controller.
 func (c Connection) ListModels(ctx context.Context) ([]base.UserModel, error) {
 	return modelmanager.NewClient(&c).ListModels("admin")