getappmap / getappmap/appmap-js

Azure backend not configured like it's documented

Open
#2,190 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
TypeScript
Stars
53
Forks
18
Avg merge
3d 18h
Merged PRs (30d)
8

Description

https://github.com/getappmap/appmap-js/blob/ec341a892c2dfe63e39cc94eb77fccadb96c0ac4/packages/cli/src/rpc/llmConfiguration.ts#L24-L42
doesn't match the documentation at https://appmap.io/docs/using-navie-ai/bring-your-own-model.html#bring-your-own-model-byom

The code only actives the Azure backend if the base URL is defined.
The code checks AZURE_OPENAI_INSTANCE_NAME, but the documentation mentions AZURE_OPENAI_API_INSTANCE_NAME.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in packages/cli/src/rpc/llmConfiguration.ts at lines 24-42 and compare the Azure environment-variable handling with the BYOM documentation page. Make the documented Azure configuration and the CLI behavior consistent, then verify that the documented setup activates the backend as expected.

Written by the indexing model from the issue text.

Assessment

Tech stack
azure, typescript
Domain
backend, cloud
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.