fix: GetOwnerID returns DAO name instead of numeric group ID

hTrap · claude · hTrap · commit 262370c4e81b · 2026-04-02T15:09:46.000+05:30
When active DAO is set, create_repo failed with "owner id must consist
minimum 3 chars" because GetOwnerID returned ActiveDAO.ID (the numeric
cosmos group ID, e.g. "3") instead of ActiveDAO.Name (e.g. "Gitopia").

Fixes Gitopia/gitopia-mcp-server#1

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/internal/session/user_context.go b/internal/session/user_context.go
@@ -106,7 +106,7 @@ func (cm *ContextManager) GetOwnerID() string {
 	}
 
 	if cm.context.ActiveDAO != nil {
-		return cm.context.ActiveDAO.ID
+		return cm.context.ActiveDAO.Name
 	}
 
 	return cm.context.Username
diff --git a/internal/session/user_context_test.go b/internal/session/user_context_test.go
@@ -54,7 +54,7 @@ func TestGetOwnerID_WithActiveDAO(t *testing.T) {
 		DAOs:      []DAO{dao},
 		ActiveDAO: &dao,
 	})
-	assert.Equal(t, "dao-123", cm.GetOwnerID())
+	assert.Equal(t, "myorg", cm.GetOwnerID())
 }
 
 func TestGetUsername_NilContext(t *testing.T) {

Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ func (cm *ContextManager) GetOwnerID() string {`
`106`	`106`	`}`
`107`	`107`
`108`	`108`	`if cm.context.ActiveDAO != nil {`
`109`		`- return cm.context.ActiveDAO.ID`
	`109`	`+ return cm.context.ActiveDAO.Name`
`110`	`110`	`}`
`111`	`111`
`112`	`112`	`return cm.context.Username`
Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ func TestGetOwnerID_WithActiveDAO(t *testing.T) {`
`54`	`54`	`DAOs: []DAO{dao},`
`55`	`55`	`ActiveDAO: &dao,`
`56`	`56`	`})`
`57`		`- assert.Equal(t, "dao-123", cm.GetOwnerID())`
	`57`	`+ assert.Equal(t, "myorg", cm.GetOwnerID())`
`58`	`58`	`}`
`59`	`59`
`60`	`60`	`func TestGetUsername_NilContext(t *testing.T) {`