Skip to content
This repository was archived by the owner on Apr 28, 2026. It is now read-only.

ENG-8869: Add inkeep - #1756

Merged
Kastier1 merged 1 commit into
mainfrom
carlos/add-inkeep
Feb 19, 2026
Merged

Kastier1 merged 1 commit into
mainfrom
carlos/add-inkeep

Conversation

@carlosabadia

Copy link
Copy Markdown
Collaborator

No description provided.

@linear

linear Bot commented Feb 19, 2026

Copy link
Copy Markdown

@greptile-apps

greptile-apps Bot commented Feb 19, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR updates the Inkeep search integration by upgrading the library version from 0.5.55 to 0.5.115 and significantly refines the styling. The changes switch the navbar search from Typesense to Inkeep, add a custom logo avatar for the AI assistant, and update the search icon SVG.

Key changes:

  • Library version bump to @inkeep/cxkit-react@0.5.115
  • Extensive CSS refinements for search bar appearance in light/dark themes
  • Added aiAssistantAvatar pointing to /logos/small_logo.svg
  • New small logo SVG asset created
  • Removed Google Fonts link (font now assumed to be available)
  • Updated search icon with simpler markup

Minor issues:

  • Duplicate CSS hover rules (lines 199-203 and 222-226)
  • Hardcoded API key could be extracted to constant

Confidence Score: 4/5

  • This PR is safe to merge with minor style improvements recommended
  • The changes are primarily cosmetic (styling updates and library version bump). The duplicate CSS rules are non-breaking and the hardcoded API key appears to be a public Inkeep search key. No functional logic changes that could introduce bugs.
  • pcweb/components/docpage/navbar/inkeep.py has duplicate CSS rules that could be cleaned up

Important Files Changed

Filename Overview
assets/logos/small_logo.svg New small logo SVG added for Inkeep avatar
pcweb/components/docpage/navbar/search.py Switched from typesense to inkeep search implementation
pcweb/components/docpage/navbar/inkeep.py Updated Inkeep library version and extensive styling changes; contains duplicate CSS rules

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[search_bar component] --> B[inkeep function]
    B --> C[Search component]
    C --> D[InkeepSearchBar component]
    D --> E[inkeep library v0.5.115]
    
    F[baseSettings] --> D
    G[searchSettings] --> D
    H[aiChatSettings] --> D
    
    F --> I[API Key]
    F --> J[Custom Search Icon]
    F --> K[Theme & CSS Styles]
    
    H --> L[AI Avatar Logo]
    H --> M[Example Questions]
    H --> N[Confidence Tool]
    
    J -.-> O[search.svg]
    L -.-> P[small_logo.svg]
    
    style D fill:#6e56cf,color:#fff
    style E fill:#e0e1e6
    style I fill:#fff4e6
Loading

Last reviewed commit: cdd81a5

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

4 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

Comment thread pcweb/components/docpage/navbar/inkeep.py
Comment thread pcweb/components/docpage/navbar/inkeep.py
@Kastier1
Kastier1 merged commit f345d0c into main Feb 19, 2026
10 checks passed
@Kastier1
Kastier1 deleted the carlos/add-inkeep branch February 19, 2026 17:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants