Skip to content

fix(secops): include entity values in related entities formatter - #280

Open
dandye wants to merge 1 commit into
mainfrom
kenn-forge/issue-268-secops-mcp-lookup-entity-formatter-bug
Open

fix(secops): include entity values in related entities formatter#280
dandye wants to merge 1 commit into
mainfrom
kenn-forge/issue-268-secops-mcp-lookup-entity-formatter-bug

Conversation

@dandye

@dandye dandye commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Fixes #268

Problem

The lookup_entity formatter was only displaying the entity type for related entities, omitting the actual entity values (IPs, domains, hashes, etc.).

Solution

Added extraction and display of entity.value alongside entity type for related entities, providing complete information about related entities in the summary.

Changes

  • Initialize entity_value default to 'Unknown'
  • Extract entity.value attribute if available
  • Update output format to include both value and type

Verification

  • Python syntax check: ✓ Passed
  • Code review: ✓ No issues found
  • Minimal change focused on bug fix

The lookup_entity formatter was only displaying the entity type for related
entities, omitting the actual entity values (IPs, domains, hashes, etc.).

This fix adds the entity value to the output alongside the type, providing
complete information about related entities in the summary.

Changes:
- Initialize entity_value default to 'Unknown'
- Extract entity.value attribute if available
- Update output format to include both value and type
@dandye
dandye requested a review from a team August 13, 2026 13:33
@google-cla

google-cla Bot commented Aug 13, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

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.

secops_mcp_lookup_entity_formatter_bug

2 participants