Skip to content

Config translator and explainer model#39

Merged
curtisman merged 1 commit intomicrosoft:mainfrom
curtisman:model
Aug 22, 2024
Merged

Config translator and explainer model#39
curtisman merged 1 commit intomicrosoft:mainfrom
curtisman:model

Conversation

@curtisman
Copy link
Copy Markdown
Member

@curtisman curtisman commented Aug 22, 2024

Enable shell and cli to config model that is used for translation and explanation.
Use command @config model set <kind> <model> to set the model for kind = translation or explanation
Use command @config model show [<kind>] to show the current set model (and the URL for the model)

Also add command to enable/disable an agent (i.e. both translation and action for an agent) using @config agent <agentname>

@curtisman curtisman enabled auto-merge August 22, 2024 23:29
@curtisman curtisman added this pull request to the merge queue Aug 22, 2024
Merged via the queue into microsoft:main with commit 0603126 Aug 22, 2024
@curtisman curtisman deleted the model branch August 22, 2024 23:49
github-merge-queue Bot pushed a commit that referenced this pull request Apr 8, 2026
…te split (#2165)

`test:local` was still picking up tests that depend on external services
and API keys via runtime gating (`testIf`/`describeIf`). This PR
reclassifies those suites so service-dependent tests run under
`test:live` instead of local runs.

- **Reclassify service-dependent suites to live**
- Renamed fully API-key-gated test files from `*.spec.ts` to `*.test.ts`
in:
    - `packages/aiclient`
    - `packages/knowPro`
    - `packages/knowledgeProcessor`
    - `packages/memory/conversation`
    - `packages/memory/storage`
    - `packages/memory/website`
- Result: these suites are no longer selected by local `*.spec`
patterns.

- **Add explicit live test scripts in affected packages**
- Added `test:live` scripts with `--testPathPattern=".*[.]test[.]js"` to
each affected package so workspace-level `pnpm run test:live` includes
them.

- **Align website online alias with live routing**
  - Updated `packages/memory/website/package.json`:
- `test:online` now delegates to `test:live` (instead of `test:local`)
to match `.online` suite intent.

```json
{
  "scripts": {
    "test:local": "pnpm run jest-esm --testPathPattern=\".*[.]spec[.]js\"",
    "test:live": "pnpm run jest-esm --testPathPattern=\".*[.]test[.]js\""
  }
}
```

> [!WARNING]
>
> <details>
> <summary>Firewall rules blocked me from connecting to one or more
addresses (expand for details)</summary>
>
> #### I tried to connect to the following addresses, but was blocked by
firewall rules:
>
> - `ip-api.com`
> - Triggering command: `/opt/hostedtoolcache/node/24.14.1/x64/bin/node
/opt/hostedtoolcache/node/24.14.1/x64/bin/node --no-warnings
--experimental-vm-modules
/home/REDACTED/work/TypeAgent/TypeAgent/ts/node_modules/.pnpm/jest-worker@29.7.0/node_modules/jest-worker/build/workers/processChild.js
Agen�� ignore src/preload dules/.bin/sh electron.vite.prsh --logLevel
error node k/Ty�� /bin/jest.js
&#39;--testPathPattern=.*[.]spec[.]js&#39; 1 dules/.bin/sh gnore dist r
uname` (dns block)
> - Triggering command: `/opt/hostedtoolcache/node/24.14.1/x64/bin/node
/opt/hostedtoolcache/node/24.14.1/x64/bin/node --no-warnings
--experimental-vm-modules
/home/REDACTED/work/TypeAgent/TypeAgent/ts/node_modules/.pnpm/jest-worker@29.7.0/node_modules/jest-worker/build/workers/processChild.js
/hom�� build src/preload k/_temp/ghcca-node/node/bin/node
electron.vite.prsh --logLevel .bin/node node k/Ty�� /bin/jest.js
&#39;--testPathPattern=.*[.]spec[.]js&#39; 1 dules/.bin/sh gnore dist
ct/node_modules/.bin/node node` (dns block)
> - Triggering command: `/opt/hostedtoolcache/node/24.14.1/x64/bin/node
/opt/hostedtoolcache/node/24.14.1/x64/bin/node --no-warnings
--experimental-vm-modules
/home/REDACTED/work/TypeAgent/TypeAgent/ts/node_modules/.pnpm/jest-worker@29.7.0/node_modules/jest-worker/build/workers/processChild.js
Agen�� build src/preload uid-build electron.vite.prsh --logLevel error
node Agen�� /bin/jest.js &#39;--testPathPattern=.*[.]spec[.]js&#39; 1
odules/pnpm/dist/node-gyp-bin/sh src/**/*.txt dist in/copyfiles node`
(dns block)
> - `www.electronjs.org`
> - Triggering command: `/opt/hostedtoolcache/node/24.14.1/x64/bin/node
/opt/hostedtoolcache/node/24.14.1/x64/bin/node
/home/REDACTED/work/TypeAgent/TypeAgent/ts/node_modules/.pnpm/@electron&#43;rebuild@4.0.3/node_modules/@electron/rebuild/lib/module-type/node-gyp/worker.js`
(dns block)
>
> If you need me to access, download, or install something from one of
these locations, you can either:
>
> - Configure [Actions setup
steps](https://gh.io/copilot/actions-setup-steps) to set up my
environment, which run before the firewall is enabled
> - Add the appropriate URLs or hosts to the custom allowlist in this
repository's [Copilot coding agent
settings](https://github.com/microsoft/TypeAgent/settings/copilot/coding_agent)
(admins only)
>
> </details>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: curtisman <12101885+curtisman@users.noreply.github.com>
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