Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move command requests to card message matrix events and add support for multiple commands per message #2145

Merged
merged 25 commits into from
Mar 10, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
b40c860
Convert tool calls to CommandRequests on boxel message events, and su…
lukemelia Feb 5, 2025
c7a2bd6
WIP Update host types for changes to how command requests are encoded
lukemelia Feb 13, 2025
e927512
Add aibot test coverage for handling multiple tool calls in one response
lukemelia Feb 17, 2025
899da9b
aibot should not respond to CommandResultEvent until all tool calls h…
lukemelia Feb 17, 2025
69acba1
Merge branch 'return-of-the-skill-commands' into cs-7993-support-mult…
lukemelia Feb 26, 2025
66cc8cc
Merge branch 'main' into cs-7993-support-multiple-tool-calls-per-ai-r…
lukemelia Feb 26, 2025
622d887
Fix aibot tests
lukemelia Feb 26, 2025
5d4dcaf
Fix and improve test of executing command that came from a skill card
lukemelia Feb 26, 2025
b9bcbec
host app support for multiple command requests in one message
lukemelia Feb 27, 2025
bbbf2c4
Before trying to load command code refs, we need to ensure they are u…
lukemelia Feb 27, 2025
9979d51
Merge branch 'main' into cs-7993-support-multiple-tool-calls-per-ai-r…
lukemelia Mar 3, 2025
6a87a81
WIP retry failed command
lukemelia Mar 3, 2025
af729d4
Retrying failed commands and failed command state
lukemelia Mar 4, 2025
264eebc
Move commandRequestId out of data node
lukemelia Mar 4, 2025
141eb81
Rename and move CommandRequestContent interface
lukemelia Mar 4, 2025
3911dc8
Minor tweaks
lukemelia Mar 4, 2025
3ad64b3
Merge branch 'main' into cs-7993-support-multiple-tool-calls-per-ai-r…
lukemelia Mar 4, 2025
44bfe64
Defensive check for attachedCardEventIds for backwards compat.
lukemelia Mar 4, 2025
1ff046c
Fix real-world streaming
lukemelia Mar 5, 2025
bdfcfc2
Restore scroll into view behavior
lukemelia Mar 5, 2025
c5ec57a
Fix assertion message
lukemelia Mar 5, 2025
53a898d
Adopt custom rel_type for command results
lukemelia Mar 6, 2025
0d84b50
Fix how command results are added to prompt when there is no body
lukemelia Mar 6, 2025
8d2d4c6
Add coverage for responder.ensureThinkingMessageSent behavior
lukemelia Mar 6, 2025
8b4e305
Merge branch 'main' into cs-7993-support-multiple-tool-calls-per-ai-r…
lukemelia Mar 6, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
33 changes: 33 additions & 0 deletions packages/ai-bot/.eslintrc.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,33 @@
'use strict';

module.exports = {
root: true,
parser: '@typescript-eslint/parser',
parserOptions: {
ecmaVersion: 2020,
sourceType: 'module',
ecmaFeatures: {
legacyDecorators: true,
},
},
plugins: ['ember'],
extends: [
'eslint:recommended',
'plugin:@typescript-eslint/recommended',
'plugin:prettier/recommended',
],
rules: {
// this doesn't work well with the monorepo. Typescript already complains if you try to import something that's not found
'import/no-unresolved': 'off',
'prefer-const': 'off',
'@typescript-eslint/ban-ts-comment': 'off',
'@typescript-eslint/ban-types': 'off',
'@typescript-eslint/prefer-as-const': 'off',
'@typescript-eslint/no-explicit-any': 'off',
'@typescript-eslint/no-non-null-assertion': 'off',
'@typescript-eslint/no-unused-vars': [
'error',
{ argsIgnorePattern: '^_', varsIgnorePattern: '^_' },
],
},
};
Loading
Loading