Commit Graph

3124 Commits

Author SHA1 Message Date
Jason Zimdars 362c174b8b Try a light theme for the fizzy bar 2025-07-25 10:27:55 -05:00
Jason Zimdars 0f54a08569 Adjust loading state so to reduce jumping 2025-07-25 10:13:00 -05:00
Jason Zimdars 3479129721 Update prompt 2025-07-25 10:07:49 -05:00
Jason Zimdars ad7d7bbe89 Revert prompt tweaks, they cause test failures 2025-07-25 07:30:02 -05:00
Jason Zimdars 919741da61 Merge branch 'main' into event-summary-tweaks
* main:
  Make output scrollable
  Dialog target needs to be in the DOM on connect
  Remove debug
  Update badge counts clientside because we can't count on pushes to remove badges
2025-07-25 07:24:15 -05:00
Jason Zimdars af99a466c0 Merge branch 'main' into update-badge-counts
* main:
  Skip failing activity summary test
  Add missing test
  Little refactor + missing test
  Add comment
  Regenerate test's fixtures to see if that fixes it for JZ
  Make output scrollable
  Dialog target needs to be in the DOM on connect
2025-07-25 07:22:54 -05:00
Jason Zimdars 85aa0465ac Merge branch 'main' into fix-dialog-controller-error
* main:
  Skip failing activity summary test
  Add missing test
  Little refactor + missing test
  Add comment
  Regenerate test's fixtures to see if that fixes it for JZ
  Make output scrollable
2025-07-25 07:21:57 -05:00
Jason Zimdars 4e935e4800 Merge branch 'main' into scrollable-command-output
* main:
  Skip failing activity summary test
  Add missing test
  Little refactor + missing test
  Add comment
  Regenerate test's fixtures to see if that fixes it for JZ
2025-07-25 07:20:52 -05:00
Jason Zimdars 9707f19b87 Merge branch 'main' into event-summary-tweaks
* main:
  Skip failing activity summary test
  Add missing test
  Little refactor + missing test
  Add comment
  Regenerate test's fixtures to see if that fixes it for JZ
2025-07-25 07:17:15 -05:00
Jason Zimdars 2ca156b2e4 Shorter word count 2025-07-25 07:17:08 -05:00
Jorge Manrubia 711d9207ed Little refactor + missing test 2025-07-25 10:04:07 +02:00
Jorge Manrubia fbf003d7ec Add comment 2025-07-25 06:32:40 +02:00
Jason Zimdars 6186c0fd63 Tweak prompt 2025-07-24 17:45:54 -05:00
Jason Zimdars b9e1bf2447 Adjust styles so summaries feel more connected to the day they describe 2025-07-24 17:40:44 -05:00
Jason Zimdars f15fac94d9 Make output scrollable 2025-07-24 17:09:56 -05:00
Jason Zimdars d3a01cf9bb Dialog target needs to be in the DOM on connect
Moves it out of the turbo frame so it's available even when empty
2025-07-24 14:58:52 -05:00
Jason Zimdars 1d6a60cd17 Remove debug 2025-07-24 14:49:53 -05:00
Jason Zimdars 8c9d7d9d91 Update badge counts clientside because we can't count on pushes to remove badges 2025-07-24 14:43:51 -05:00
Jorge Manrubia 18597aabad Make sure we sort closed cards by recency by default
So that the filter used for the insight context fetches the most recent cards when using indexed_by closed
2025-07-24 17:46:40 +02:00
Jorge Manrubia 95e065a0c4 Reduce the injected load a bit 2025-07-24 17:33:40 +02:00
Jorge Manrubia 523f3f0acb Set some hard limits on the prompt lenghts we inject 2025-07-24 17:32:50 +02:00
Jorge Manrubia 0429dc28f3 Fix for injecting the current context
This is a workaround to make the prototype work. We create a context on the fly to execute AI-derived
commands. When those require listing cards, we ended up losing the notion of "being on a perma screen".
2025-07-24 17:27:32 +02:00
Jorge Manrubia c2120fe83d Remove results when pressing up 2025-07-24 16:58:36 +02:00
Jorge Manrubia b2ee79bbd0 Format 2025-07-24 16:48:06 +02:00
Jorge Manrubia e61ee25b47 Suite passing 2025-07-24 16:48:05 +02:00
Jorge Manrubia 4574c0d85c Include stage information about the card 2025-07-24 16:48:05 +02:00
Jorge Manrubia ba97ec05d6 Remove transaction as the AI commands can hold the transaction for too long 2025-07-24 16:48:05 +02:00
Jorge Manrubia 5ad2d09eae Move the hack to fetch completed cards to the insight command logic
We need to rework this as part of the filters revamp. "indexed_by" being
a mix of filters keeps biting us.
2025-07-24 16:48:05 +02:00
Jorge Manrubia fc26329daf Restore default to open cards. Use special "all" value instead.
The change could have side effects (e.g: when running commands).

This also include further changes.
2025-07-24 16:48:04 +02:00
Jorge Manrubia 018c746d2a Make the prompt smarter about the current context to make better calls 2025-07-24 16:48:03 +02:00
Jorge Manrubia 8bb7783215 Extract "prompt generation" logic to models for events, comments and cards
So that we can reuse in other scenarios
2025-07-24 16:48:03 +02:00
Jorge Manrubia dd28e3d1e9 Replace special value with just not asuming that filters are about open cards by default 2025-07-24 16:48:03 +02:00
Jorge Manrubia 9d0887f6bd Use a workaround with a special indexed_by filter to "query all" 2025-07-24 16:48:02 +02:00
Jorge Manrubia 83d743fe2a Make the translator use the /insight command 2025-07-24 16:48:02 +02:00
Jorge Manrubia 7a8bf4895c More prompt fixes 2025-07-24 16:48:01 +02:00
Jorge Manrubia 4d3e7bf0af Fix: llm model 2025-07-24 16:48:01 +02:00
Jorge Manrubia 1c6adf54a8 Restore semantic searches 2025-07-24 16:47:57 +02:00
Jorge Manrubia df9c1e0674 Make sure we pass expressions as a single term
Multiple terms refine the search on each term and works way
worse with the semantic approach
2025-07-24 16:47:54 +02:00
Jorge Manrubia 6080c2aa9b Refine when to filter by cards, make it more relaxes with the expressions it accepts 2025-07-24 16:47:53 +02:00
Jason Zimdars 8febb2b3a1 Merge pull request #804 from basecamp/fix-app-icons
App icons should have backgrounds
2025-07-24 09:31:09 -05:00
Jason Zimdars 50d5955be3 App icons should have backgrounds
Also: create a proper, padded maskable version for Android
2025-07-24 09:27:23 -05:00
Stanko K.R. 66a8d35060 Fix typo in the card recovery message 2025-07-24 15:26:26 +02:00
Jason Zimdars 3a53d8359c This didn't update the page because it never hit the format.html render 2025-07-23 18:32:02 -05:00
Jason Zimdars be9c4b706f This needs to always be present to so we can mark unread when there are no unreads 2025-07-23 17:57:12 -05:00
Jason Zimdars 046d66fbed Consistent with Mark as read 2025-07-23 17:45:12 -05:00
Jason Zimdars 250d38768f Rename helper since it now handles both actions 2025-07-23 17:44:52 -05:00
Jason Zimdars db92de7a27 data-controller = form needs to be on the form or it causes errors
- The whole reason we're using `form.submit` here is to prevent clicks on the card link
2025-07-23 17:43:15 -05:00
Jason Zimdars 99df4b1a12 No longer used 2025-07-23 17:33:56 -05:00
Jason Zimdars 43e515f2a7 Always display the mark-as-unread button in previously seen 2025-07-23 17:32:07 -05:00
Jason Zimdars 4276aad5a7 Mark notifications as unread from the index 2025-07-23 17:01:59 -05:00