Skip to content

AI Tour Tokyo updates #340

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

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

cedricvidal
Copy link
Contributor

@cedricvidal cedricvidal commented Mar 26, 2025

Purpose

Optimizations

Init researcher agent at startup time

Moved to startup time:

  • AI Project connection
  • AI project init
  • Bing connection
  • Bing grounding tool
  • Extract instructions from system message in Prompty template
  • Agent init
  • Reuse agent if it already exists instead of creating a new one
  • Moved researcher feedback to separate assistant message in Prompty template
  • Keeping the AI Project opened as we're reusing the AI project between requests

Others

  • Bumped TPM to 150
  • Prompty file doesn't need to be loaded twice

Bug fixes

  • feedback from editor is now passed to researcher agent
  • Add --host to vite in dev mode in order to work in dev container

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type

What kind of change does this Pull Request introduce?

[x] Bugfix
[ ] Feature
[x] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

In order to work in dev container
Moved to startup time:
- AI Project connection
- AI project init
- Bing connection
- Bing grounding tool
- Extract instructions from system message in Prompty template
- Agent init
- Reuse agent if it already exists instead of creating a new one
- Moved researcher feedback to separate assistant message in Prompty template

Bug fixes:
- feedback from editor is now passed to researcher agent
We're reusing the AI project between requests
@cedricvidal cedricvidal marked this pull request as draft March 26, 2025 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant