Commit Graph

621 Commits

Author SHA1 Message Date
Jason Zimdars aa13853ae4 Add a third, "On deck" column in Collections 2025-08-12 17:58:12 -05:00
Jason Zimdars e8dc14fda7 Simplify prompt, adjust styles to move away from bullet format 2025-08-11 18:03:48 -05:00
Mike Dalessio d2da1dcc06 Make ExcerptHelper work for String inputs 2025-08-11 17:38:24 -04:00
Mike Dalessio d2a0196fe2 Backfill tests for ExcerptHelper 2025-08-11 17:33:19 -04:00
Jorge Manrubia 588f03e927 Remove insight experimental code
Also, downgrade LLM for commands to mini. It's much cheaper and I only upgraded because of the
interpretation complexities that the insight command introduced.

We are implementing a much better approach with Ask.
2025-08-11 13:40:56 +02:00
Jorge Manrubia 8addd4bd50 Remove comment 2025-08-11 12:42:31 +02:00
Mike Dalessio 5540b0fd1b Make sure session cookie has the correct path for the tenant
ref: https://fizzy.37signals.com/5986089/collections/7/cards/1234
2025-08-10 18:53:52 -04:00
Mike Dalessio 360e22feb0 Move testing setup for path-based tenanting into test helper
This was previously in the AR::Tenanted branch we were using, but
feels like it doesn't belong in the gem.
2025-08-10 18:53:33 -04:00
Mike Dalessio 9bdc273c76 Prefer Account#slug to Account.script_name
Caching the script name in a class instance variable seemed like a bug
to me.
2025-08-09 14:27:41 -04:00
Mike Dalessio 57269b1b9c Allow local authentication with LOCAL_AUTHENTICATION=1
You can touch `tmp/local-auth.txt` to persist this setting.
2025-08-06 17:05:35 -04:00
Jason Zimdars 9362b711d8 Adjust prompt 2025-08-05 16:02:11 -05:00
Jason Zimdars b30a329a1a Surface some fun user activity 2025-07-31 14:30:08 -05:00
Jason Zimdars 7d55069d88 Adjust prompt 2025-07-31 10:30:00 -05:00
Jason Zimdars b795a43d5c Simplify the prompt, try a top 5 points format 2025-07-30 18:14:37 -05:00
Jason Zimdars 0f3aa31943 Bold names, ensure summaries generate correctly structured collection links 2025-07-29 11:57:23 -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 07a470a8ac Update tests 2025-07-28 13:12:53 -05:00
Jason Zimdars 3479129721 Update prompt 2025-07-25 10:07:49 -05:00
Jorge Manrubia 539f83eaee Add VCR fixtures for test 2025-07-25 14:51:28 +02:00
Kevin McConnell 02bdfc780c Skip failing activity summary test 2025-07-25 11:45:22 +01:00
Jorge Manrubia d90fac853f Add missing test 2025-07-25 10:09:00 +02:00
Jorge Manrubia 711d9207ed Little refactor + missing test 2025-07-25 10:04:07 +02:00
Jorge Manrubia d5bedd812d Regenerate test's fixtures to see if that fixes it for JZ 2025-07-25 06:27:12 +02:00
Andy Smith 8e795f183d Update system message to highlight verbs 2025-07-24 13:42:19 -05:00
Jorge Manrubia c128b07203 Add test 2025-07-24 17:47:42 +02:00
Jorge Manrubia 7de45cc061 Remove vcr_record! and fix VCR fixtures due to different timestamps 2025-07-24 16:48:06 +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 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 1c6adf54a8 Restore semantic searches 2025-07-24 16:47:57 +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 84f19183be Add model test for read and unread 2025-07-23 17:28:15 -05:00
Jason Zimdars 6ef390f201 Test the destroy action 2025-07-23 17:26:01 -05:00
Jason Zimdars dfb6198e94 Merge branch 'main' into web-push
* main: (50 commits)
  Regenerate fixtures
  Let summaries know about collections
  Regenerate fixtures
  More tweaks
  Regenerate fixtures after tweaking prompt
  Fix model selection (I had broken it!)
  Some prompt refinements
  Some of us have basecamp.com email addresses
  Update nokogiri and thor to addres CVEs
  Fix tests
  Format
  Move model options to helper
  Tidy up the admin tool for prompts
  Extract methods
  Format
  Default to the current summary
  Make models selectable
  Add an animated effect for the generating state
  Remove debug
  Wrap them both in the container so the generating state looks right
  ...
2025-07-23 11:50:35 -05:00
Jorge Manrubia 6a2614d025 Regenerate fixtures 2025-07-23 16:58:06 +02:00
Jorge Manrubia 62994c05f8 Regenerate fixtures 2025-07-23 13:18:36 +02:00
Jorge Manrubia da4afe7252 Regenerate fixtures after tweaking prompt 2025-07-23 13:12:41 +02:00
Jorge Manrubia fc1ff7bf5e Fix tests 2025-07-23 12:21:20 +02:00
Jorge Manrubia c3ae05833e Make models selectable 2025-07-23 11:02:34 +02:00
Jorge Manrubia e6004047fc Rename file 2025-07-22 19:14:07 +02:00
Jorge Manrubia 493d517a0c Make sure non-staff user can't access to the prompt editor 2025-07-22 19:05:59 +02:00
Jorge Manrubia fd0484176c Format 2025-07-22 18:59:10 +02:00
Jorge Manrubia 2a7552f474 Add controller test 2025-07-22 18:57:50 +02:00
Jorge Manrubia 70f19e54b5 Add assertion on HTML content 2025-07-22 18:47:48 +02:00