Add console agent sample #248

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.

Sign up for GitHub

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

Jump to bottom

Open

stevkan wants to merge 11 commits into microsoft:main from stevkan:add-console-agent-sample

+1,114 −0

Contributor

stevkan commented Aug 8, 2025

No description provided.

stevkan added 5 commits

July 31, 2025 15:57


          Adds the console echo agent sample

275e70a


          Fix console-echo-agent name in package-lock.json and ensured proper a…

df72d35

…sync handling in index.ts


          Removed to re-align with new samples folder structure

c070e07


          Merge branch 'main' of https://github.com/microsoft/Agents into add-c…

b5ce27b

…onsole-agent-sample


          Adds the console echo agent sample

9d41d0e

github-actions bot added Samples From Fork labels

rido-min requested changes

View reviewed changes

samples/nodejs/console-echo/package.json Outdated Show resolved Hide resolved

samples/nodejs/console-echo/src/index.ts Outdated Show resolved Hide resolved

samples/nodejs/console-echo/README.md Outdated Show resolved Hide resolved

rido-min requested changes

View reviewed changes

Member

rido-min left a comment

there are lint issues failing the checks.


          simplify console adapter

b5cbb86

rido-min reviewed

View reviewed changes

samples/nodejs/console-echo/src/dialogAgent.ts Outdated Show resolved Hide resolved

rido-min reviewed

View reviewed changes

samples/nodejs/console-echo/src/consoleAdapter.ts Show resolved Hide resolved

rido-min reviewed

View reviewed changes

samples/nodejs/console-echo/src/consoleAdapter.ts Outdated Show resolved Hide resolved

rido-min reviewed

View reviewed changes

samples/nodejs/console-echo/src/consoleAdapter.ts Outdated Show resolved Hide resolved

rido-min reviewed

View reviewed changes

samples/nodejs/console-echo/src/consoleAdapter.ts Outdated Show resolved Hide resolved

Member

rido-min commented Aug 18, 2025

@stevkan I have fixed the lint issues, and applyed my suggestions here b5cbb86

stevkan added 2 commits

August 18, 2025 12:14


          Merge branch 'main' of https://github.com/microsoft/Agents into add-c…

30331bf

…onsole-agent-sample


          Merge remote-tracking branch 'upstream/fix/248-rido' into add-console…

5ca99ba

…-agent-sample

Contributor Author

stevkan commented Aug 18, 2025

Will submit an updated PR with the suggested changes.


          Refactor README and ConsoleAdapter comments for clarity and conciseness

904bdcd

rido-min approved these changes

View reviewed changes

rido-min requested changes

View reviewed changes

Member

rido-min left a comment

there are some issues with the linter

MattB-msft and others added 2 commits

August 19, 2025 16:55


          Merge branch 'main' into add-console-agent-sample

71da2c6


          Merge branch 'main' into add-console-agent-sample

fcef273

rido-min approved these changes

View reviewed changes

rido-min requested changes

View reviewed changes

Member

rido-min left a comment

there are some linting issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

From Fork Samples