feat: cache per owner installation tokens + retry improvements

Shubham Hibare · Shubham Hibare · commit f5d7b6cd66a6 · 2026-05-01T23:37:23.000+05:30
diff --git a/internal/clients/github/client_test.go b/internal/clients/github/client_test.go
@@ -663,8 +663,7 @@ func TestGetContents_RepoOutsideInstallationScope(t *testing.T) {
 	client.installations.set("example-org", 12345)
 
 	_, err := c.GetContents(t.Context(), "example-org/excluded-repo", ".github/trust-policy.yaml")
-	require.Error(t, err)
-	assert.ErrorIs(t, err, ErrRepositoryNotFound)
+	require.ErrorIs(t, err, ErrRepositoryNotFound)
 	assert.Equal(t, int32(0), installationLookups.Load(), "warm installation cache should skip lookup")
 }
 

Original file line number	Diff line number	Diff line change
`@@ -663,8 +663,7 @@ func TestGetContents_RepoOutsideInstallationScope(t *testing.T) {`
`663`	`663`	`client.installations.set("example-org", 12345)`
`664`	`664`
`665`	`665`	`_, err := c.GetContents(t.Context(), "example-org/excluded-repo", ".github/trust-policy.yaml")`
`666`		`- require.Error(t, err)`
`667`		`- assert.ErrorIs(t, err, ErrRepositoryNotFound)`
	`666`	`+ require.ErrorIs(t, err, ErrRepositoryNotFound)`
`668`	`667`	`assert.Equal(t, int32(0), installationLookups.Load(), "warm installation cache should skip lookup")`
`669`	`668`	`}`
`670`	`669`