Commit Graph

3137 Commits

Author SHA1 Message Date
Andy Smith 5fa6e50ecb Merge pull request #806 from basecamp/subtle-closed-stamp
Use light green color for stamp
2025-07-28 15:17:35 -05:00
Jason Zimdars 2da9d1893c Merge branch 'adjust-fizzy-bar'
* adjust-fizzy-bar:
  Remove input label, adjust placeholder, and text size
2025-07-28 15:04:23 -05:00
Jason Zimdars 7b021ac192 Remove input label, adjust placeholder, and text size 2025-07-28 15:02:42 -05:00
Andy Smith ece1751538 Merge pull request #805 from basecamp/bold-event-actions
Update system message to highlight verbs
2025-07-28 13:23:34 -05:00
Andy Smith c664040c2d Update stamp style 2025-07-28 13:07:43 -05:00
Andy Smith a213ac666e Revert sentence structure, but keep bold text 2025-07-28 12:24:49 -05:00
Jason Zimdars 938a84ea7c Use the same excerpt formatting as Latest Activity events 2025-07-28 11:22:15 -05:00
Jason Zimdars d7579fc6dc Use classes to avoid collisions
Fixes regression on prompt edit button
2025-07-28 10:59:49 -05:00
Andy Smith fd7630e8c6 Merge branch 'main' into subtle-closed-stamp
* main:
  Try a light theme for the fizzy bar
  Adjust loading state so to reduce jumping
  Update prompt
  More clarifications
  Clarify how to work with VCR
  Add VCR fixtures for test
  Revert prompt tweaks, they cause test failures
  Shorter word count
  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
  Tweak prompt
  Adjust styles so summaries feel more connected to the day they describe
  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
  Don't use fancy text wrapping when editing
2025-07-28 10:27:07 -05:00
Andy Smith 93180069c4 Merge branch 'main' into no-pretty-wrapping-in-editor
* main:
  Try a light theme for the fizzy bar
  Adjust loading state so to reduce jumping
  Update prompt
  More clarifications
  Clarify how to work with VCR
  Add VCR fixtures for test
  Revert prompt tweaks, they cause test failures
  Shorter word count
  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
  Tweak prompt
  Adjust styles so summaries feel more connected to the day they describe
  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-28 10:16:38 -05:00
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
Andy Smith 3841eee87c Don't use fancy text wrapping when editing 2025-07-24 14:25:00 -05:00
Andy Smith a90d787c47 Use light green color for stamp 2025-07-24 14:04:55 -05:00
Andy Smith 8e795f183d Update system message to highlight verbs 2025-07-24 13:42:19 -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