fix(quota): detect project from auth token and guard empty project ID

larkly · larkly · commit e327ebefbb0b · 2026-04-01T13:58:35.000+02:00
Application credentials are always scoped to a project, but the project
ID was never extracted from the auth token — the stub was left as a
TODO. This caused quota lookups to fail with a 403 because the API URL
contained an empty project ID (/os-quota-sets//detail).

Extract the project from the token via ExtractProject() so it is
pre-selected on startup, and add a guard in fetchQuotas() to return a
clear error instead of making a malformed API call.
diff --git a/src/internal/app/app.go b/src/internal/app/app.go
@@ -9,6 +9,7 @@ import (
 
 	"charm.land/bubbles/v2/key"
 	"charm.land/bubbletea/v2"
+	"github.com/gophercloud/gophercloud/v2/openstack/identity/v3/tokens"
 	"github.com/larkly/lazystack/internal/cloud"
 	"github.com/larkly/lazystack/internal/compute"
 	"github.com/larkly/lazystack/internal/config"
@@ -1081,11 +1082,14 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 				for _, p := range projs {
 					infos = append(infos, shared.ProjectInfo{ID: p.ID, Name: p.Name})
 				}
-				// Try to get current project ID from the auth scope
+				// Extract current project ID from the auth token scope
 				currentID := ""
-				if pc.GetAuthResult() != nil {
-					// The token should have project scope info
-					// We'll match by checking project IDs
+				if ar, ok := pc.GetAuthResult().(interface {
+					ExtractProject() (*tokens.Project, error)
+				}); ok {
+					if proj, err := ar.ExtractProject(); err == nil && proj != nil {
+						currentID = proj.ID
+					}
 				}
 				return shared.ProjectsLoadedMsg{Projects: infos, CurrentID: currentID}
 			})
diff --git a/src/internal/ui/quotaview/quotaview.go b/src/internal/ui/quotaview/quotaview.go
@@ -94,6 +94,9 @@ func (m *Model) fetchQuotas() tea.Cmd {
 	projectID := m.projectID
 
 	return func() tea.Msg {
+		if projectID == "" {
+			return quotaLoadedMsg{err: fmt.Errorf("no project selected")}
+		}
 		ctx := context.Background()
 		var (
 			computeQuotas []quota.QuotaUsage
