Skip to content

Conversation

@FireChickenProductivity
Copy link
Contributor

@FireChickenProductivity FireChickenProductivity commented Dec 17, 2025

The goal is to make things like "settings.get()" and "actions.clip.text()" a little easier.

Copy link
Collaborator

@nriley nriley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comments from the community backlog session:

  • Use actions instead of corresponding methods where possible
  • For actions, enable user.talon_populate_lists by default for non-dragon and fix the documentation in the file that declares the actions that reference this.

@FireChickenProductivity
Copy link
Contributor Author

Comments from the community backlog session:

  • Use actions instead of corresponding methods where possible
  • For actions, enable user.talon_populate_lists by default for non-dragon and fix the documentation in the file that declares the actions that reference this.

I reorganized the talon common file. The lists are now populated by default with conformer. Actions cannot be used for settings.get without beta, so I did not make that change. We might want to do some performance testing with the populate lists tag before merge, but it works fine on my machine.

I decided to address the action command adding an unnecessary new line by only using the selected text as an argument with a wrap command variant.

@nriley
Copy link
Collaborator

nriley commented Dec 27, 2025

From the community backlog session — this is ready to merge, but we want to make sure it does not adversely affect performance on slower computers and the release version of Talon.

This is likely to cause a delay on grammar compilation which typically occurs when you issue your first voice command in a new context.

@nriley nriley added the needs testing Please test this and provide feedback in the PR! label Dec 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs testing Please test this and provide feedback in the PR!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants