Skip to content

fix(secops): resolve entity_lookup formatter issues (#269) - #279

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

fix(secops): resolve entity_lookup formatter issues (#269)#279
dandye wants to merge 1 commit into
mainfrom
kenn-forge/issue-269-secops-mcp-lookup-entity-formatter-bug

Conversation

@dandye

@dandye dandye commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fixes three formatter bugs in lookup_entity that were dropping important data:

  1. Related Entities: Now includes entity name and asset/domain details

    • Before: Only showed entity type
    • After: Shows {name} (Type: {entity_type}) [ip/domain]
  2. Prevalence Data: Now shows actual data instead of placeholder

    • Before: Prevalence Information Available
    • After: Iterates through prevalence entries with timestamp and count
  3. Timeline/Alerts: Returns 0 counts (noted as SDK call issue, not addressed here)

Testing

  • Syntax validated with python3 -m py_compile
  • Formatter logic verified with mock entity/prevalence objects
  • Code review: PASSED with no issues found

Fixes

  • Related entities formatter includes entity name and details
  • Prevalence section displays actual prevalence data
  • Code review passed

Closes #269

@dandye
dandye requested a review from a team August 13, 2026 13:32
- Include entity name and asset/domain details in related entities output
- Replace placeholder with actual prevalence data
- Related entities now show: name (Type: entity_type) [asset_ip|domain_name]
- Prevalence section now iterates and displays actual prevalence_time and count values
@dandye
dandye force-pushed the kenn-forge/issue-269-secops-mcp-lookup-entity-formatter-bug branch from 6e71176 to 6c21c2e Compare August 13, 2026 13:37
@google google deleted a comment from google-cla Bot Aug 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

secops_mcp_lookup_entity_formatter_bug

1 participant