Skip to content

Improvements to ListEntities#6535

Open
evankanderson wants to merge 1 commit into
mindersec:mainfrom
evankanderson:entity-list-utility
Open

Improvements to ListEntities#6535
evankanderson wants to merge 1 commit into
mindersec:mainfrom
evankanderson:entity-list-utility

Conversation

@evankanderson

Copy link
Copy Markdown
Member

Summary

This ended up being a few different changes to ListEntities; four are user-visible, and one is performance-related:

  • Adds a proto field indicating the "originating" entity for resources like pull requests, which can help when building a tree of related entities.
  • Reports the provider name associated with entities, which can also help with grouping
  • Adds support for listing all types of entities with a single API call
  • Adds pagination support
  • Converts the 1 + N database calls to fetch properties for entities to a single database call with additional data using JSON_OBJECT_AGG

Testing

Manually tested as well as adding integration tests (assisted by Gemini 3 Flash, but adjusted by hand).

@evankanderson evankanderson requested a review from a team as a code owner June 23, 2026 15:04
@coveralls

coveralls commented Jun 23, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 61.296% (+0.6%) from 60.703% — evankanderson:entity-list-utility into mindersec:main

"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
"go.uber.org/mock/gomock"
"golang.org/x/exp/maps"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This was fixing a lint error.

Comment thread internal/entities/service/service.go Outdated
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.

3 participants