Repository navigation
Only trigger Claude conditional prompting for 5.0 and 5.1 models - #403
Conversation
|
Preview deployed to Connect ( Deployed from commit dcde5b0. |
|
Preview root: https://posit-dev.github.io/commons/pr-403/ Python site preview: https://posit-dev.github.io/commons/pr-403/py/ Built from the latest commit on this branch. The R links in it point at the published R site, which no pull request rebuilds. |
|
Preview deployed to Connect ( Deployed from commit dcde5b0. |
There was a problem hiding this comment.
I'm wondering if we should be renaming this (and similarly in Python) now that we know that Claude 5.5+ is fine. Hard to come up with a Claude-only name, but could be needs_brevity_prompting() or something (with a comment about the Claude 5.0 and 5.1 saga). Will leave it up to you whether this ought to happen.
There was a problem hiding this comment.
We can rename! It's now confusing.
There was a problem hiding this comment.
Re-requesting review because it changed so many files
|
Cleaned up 2 preview bundle(s) on https://dogfood.team.pct.posit.it: 377641, 378009 |
|
Cleaned up 2 preview bundle(s) on https://connect.staging.pct.posit.it: 3388, 3405 |
2 participants
Closes #399
Updates the conditional prompting about brevity to only trigger for Claude 5.0 and 5.1 models, instead of all Claude 5.x models.
Adds a
skip_on_cran()test that begins failing on 2027-04-06 (6 months from now), as a reminder to remove this conditional prompting entirely once the Claude 5/5.1 models are no longer recent.