improvement(home): consolidate chat context kind icon registry

[waleedlatif1]

Summary

• Add CHAT_CONTEXT_KIND_REGISTRY keyed by ChatContextKind as the single source of truth for chip icon/label; the Record<ChatContextKind, …> typing makes it a compile error to add a new kind without an icon entry
• Fix missing book icon on logs resource chips in the home user-input
• Replace the switch in mapResourceToContext with a typed Record<MothershipResourceType, …> map; also wires the previously missing log → logs conversion so logs chips carry an executionId

Type of Change

• Bug fix
• Improvement

Testing

Tested manually — @-mention each resource type (workflow, knowledge, table, file, folder, past chat, logs) and confirmed each chip renders the correct icon.

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		May 2, 2026 5:17am

[cursor]

PR Summary

Low Risk
Low risk UI/typing refactor; main behavior change is adding explicit log→logs context mapping (with executionId) which could affect how log chips render/behave if any downstream expects the old default docs mapping.

Overview
Introduces a typed CHAT_CONTEXT_KIND_REGISTRY as the single source of truth for context-chip labels/icons (including adding the missing logs/Library icon) and updates ContextMentionIcon to render via this registry.

Refactors mapResourceToContext to a Record<MothershipResourceType, …> conversion map, and fixes the previously missing log resource mapping so log mentions become logs contexts carrying an executionId.

^{Reviewed by Cursor Bugbot for commit 13409d7. Configure here.}

[greptile-apps]

Greptile Summary

This PR consolidates chip icon/label logic into a new CHAT_CONTEXT_KIND_REGISTRY (typed Record<ChatContextKind, …>) and replaces the mapResourceToContext switch with a parallel RESOURCE_TO_CONTEXT record. Both refactors add compile-time exhaustiveness checks — adding a new ChatContextKind or MothershipResourceType will now fail compilation unless registry entries are provided.

The PR also ships two genuine bug fixes: the missing Library icon on logs chips, and the missing executionId wiring for the log → logs conversion in mapResourceToContext.

Confidence Score: 5/5

Safe to merge — clean refactor with exhaustiveness improvements and two bug fixes, no regressions identified.

All changed code is narrowly scoped to icon/label rendering and resource-to-context mapping. The Record-typed registries enforce exhaustiveness at compile time and match all current union members. No P0/P1 issues found; the previously debated runtime fallback question was thoroughly resolved in the thread.

No files require special attention.

Important Files Changed

Filename	Overview
apps/sim/app/workspace/[workspaceId]/home/components/chat-context-kind-registry.tsx	New registry file providing a typed Record<ChatContextKind, …> map for chip icons and labels; exhaustively covers all 13 ChatContextKind variants with correct renderIcon implementations.
apps/sim/app/workspace/[workspaceId]/home/components/context-mention-icon.tsx	Simplified from a switch statement to a registry lookup; delegates icon rendering to CHAT_CONTEXT_KIND_REGISTRY. Also implicitly extends coverage to workflow_block (previously fell through to default: return null).
apps/sim/app/workspace/[workspaceId]/home/components/user-input/components/constants.ts	Replaces the mapResourceToContext switch with a typed Record<MothershipResourceType, …> map; adds the previously missing log → logs conversion that now correctly wires executionId.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["ContextMentionIcon\n(context: ChatMessageContext)"] --> B["CHAT_CONTEXT_KIND_REGISTRY[context.kind]"]
    B --> C{kind}
    C --> D["workflow / current_workflow /\nworkflow_block → renderWorkflowSquare"]
    C --> E["knowledge → Database icon"]
    C --> F["table → TableIcon"]
    C --> G["file → getDocumentIcon"]
    C --> H["folder → FolderIcon"]
    C --> I["past_chat → Blimp"]
    C --> J["logs → Library icon ✅ (bug fix)"]
    C --> K["blocks / templates /\ndocs / slash_command → null"]

    L["mapResourceToContext(resource)"] --> M["RESOURCE_TO_CONTEXT[resource.type]"]
    M --> N{MothershipResourceType}
    N --> O["workflow → kind:'workflow'"]
    N --> P["knowledgebase → kind:'knowledge'"]
    N --> Q["table / file / folder"]
    N --> R["task → kind:'past_chat'"]
    N --> S["log → kind:'logs' + executionId ✅ (bug fix)"]
    N --> T["generic → kind:'docs'"]

Loading

_{Reviews (2): Last reviewed commit: "revert(home): drop unneeded mapResourceT..." | Re-trigger Greptile}

[waleedlatif1]

@greptile

[waleedlatif1]

@cursor review

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit 13409d7. Configure here.}