Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: discourse/discourse-ai
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: ai-team-sandbox
Choose a base ref
...
head repository: discourse/discourse-ai
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: main
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 9 commits
  • 14 files changed
  • 4 contributors

Commits on May 29, 2025

  1. FEATURE: simplify streaming implementation - rush last update (#1380)

    * FEATURE: simplify streaming implementation - rush last update
    
    Previous to this change we would simply "flash" the final update
    on the screen, this amends it so we quickly update the UI in about
    1 second in the end with all the final update.
    
    This makes the UI feel more interactive to end users.
    
    * DEV: Updates...
    
    - Remove unnecessary comments
    - Add doc style comments for all methods
    - Organize methods (private at bottom)
    - Update some variable names
    
    ---------
    
    Co-authored-by: Keegan George <kgeorge13@gmail.com>
    SamSaffron and keegangeorge authored May 29, 2025
    Configuration menu
    Copy the full SHA
    c06d7b0 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    59d6e2b View commit details
    Browse the repository at this point in the history

Commits on May 30, 2025

  1. FIX: Full page search broken (#1383)

    Follow-up to 59d6e2b
    tgxworld authored May 30, 2025
    Configuration menu
    Copy the full SHA
    9f43df0 View commit details
    Browse the repository at this point in the history
  2. FEATURE: support upload.getUrl in custom tools (#1384)

    * FEATURE: support upload.getUrl in custom tools
    
    Some tools need to share images with an API. A common pattern
    is for APIs to expect a URL.
    
    This allows converting upload://123123 to a proper CDN friendly
    URL from within a custom tool
    
    * no support for secure uploads, so be explicit about it.
    SamSaffron authored May 30, 2025
    Configuration menu
    Copy the full SHA
    77ae426 View commit details
    Browse the repository at this point in the history
  3. FIX: custom tools incorrectly setting all fields to blank enum (#1385)

    Previous to this change, enum was set to [] which broke all non
    enum tools
    SamSaffron authored May 30, 2025
    Configuration menu
    Copy the full SHA
    b5d393b View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    e6876aa View commit details
    Browse the repository at this point in the history
  5. UX: AI composer helper refinements (#1387)

    This update includes a variety of small refinements to the AI composer helper:
    
    - prevent height jump when going from loading text placeholder → proofreading text streaming
    - update padding on AI helper options list to be more suitable with typical Discourse menu design
    - for composer helper results that are not `showResultAsDiff` (i.e. translation):
       - update before/after diff design to be more subtle
       - results should be in normal font (as the text is cooked and not raw markdown)
    - fix: smooth streaming animation stuck showing dot icon even after smooth streaming is done
    keegangeorge authored May 30, 2025
    Configuration menu
    Copy the full SHA
    38f7e9c View commit details
    Browse the repository at this point in the history
  6. FIX: Exporting overall sentiment fails (#1388)

    ## 🔍 Overview
    
    When exporting an Overall Sentiment report in the admin panel, the export fails with:
    
    ```ruby
    Job exception: no implicit conversion of Symbol into Integer
    ```
    
    This was happening because we are passing a single _Hash_ to `report.data` however, exports expect `report.data` to be an _Array of Hashes_. This update fixes this issue by wrapping the data in an array.
    keegangeorge authored May 30, 2025
    Configuration menu
    Copy the full SHA
    34c98de View commit details
    Browse the repository at this point in the history
  7. PERF: Optimize .ai-debug-modal__tokens selector (#1390)

    This is showing as the most expensive CSS selector in Discourse at the moment. Adding specific classes and dropping the general `span` selector will make this much cheaper.
    davidtaylorhq authored May 30, 2025
    Configuration menu
    Copy the full SHA
    4ce8973 View commit details
    Browse the repository at this point in the history
Loading