Skip to content

Commit 89402a3

Browse files
chore: pre-public-release cleanup
- Remove internal document reference from models.py (personal doc path) - Remove duplicate 'Built-in -> Amplifier Capability Mapping' header in _constants.py - Fix provider name in __init__.py docstring (copilot-sdk -> github-copilot) - Clarify amplifier-core is runtime-provided in README dependencies - Broaden pyproject.toml keywords to reflect multi-model support 🤖 Generated with [Amplifier](https://github.com/microsoft/amplifier) Co-Authored-By: Amplifier <240397093+microsoft-amplifier@users.noreply.github.com>
1 parent 0bafa1a commit 89402a3

5 files changed

Lines changed: 4 additions & 9 deletions

File tree

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ make check # Full check (lint + test)
152152

153153
## Dependencies
154154

155-
- `amplifier-core>=1.0.0`
155+
- `amplifier-core` (provided by Amplifier runtime, not installed separately)
156156
- `github-copilot-sdk>=0.1.0,<0.2.0`
157157

158158

amplifier_module_provider_github_copilot/__init__.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
```yaml
2222
# In amplifier.yaml
2323
providers:
24-
copilot-sdk:
24+
github-copilot:
2525
model: claude-opus-4.5
2626
# timeout: 3600 # 1 hour default (override if needed)
2727
# thinking_timeout: 3600 # Same as regular (override if needed)
@@ -30,7 +30,7 @@
3030
3131
Or via CLI:
3232
```bash
33-
amplifier chat --provider copilot-sdk
33+
amplifier chat --provider github-copilot
3434
```
3535
3636
Prerequisites:

amplifier_module_provider_github_copilot/_constants.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -182,10 +182,6 @@ class LoopExitMethod(Enum):
182182
# Evidence: Same (delegate, report_intent) pair repeated 303 times
183183
DEDUPLICATE_TOOL_CALLS = True
184184

185-
# ═══════════════════════════════════════════════════════════════════════════════
186-
# Built-in → Amplifier Capability Mapping
187-
# ═══════════════════════════════════════════════════════════════════════════════
188-
189185
BUILTIN_TO_AMPLIFIER_CAPABILITY: dict[str, frozenset[str]] = {
190186
# File operation overlaps
191187
"view": frozenset({"read_file"}),

amplifier_module_provider_github_copilot/models.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,6 @@ def copilot_model_to_internal(raw_model: Any) -> CopilotModelInfo:
9393
# Derive max output from context window minus prompt allocation
9494
# NOTE: Do NOT cap this value. The SDK provides authoritative limits.
9595
# Capping causes incorrect budget calculation in context manager.
96-
# See: myDocs-Amp-CLI-SDK-provider/SDK-TOKEN-LIMITS-RESEARCH-20260208.md
9796
max_output_tokens = context_window - limits.max_prompt_tokens
9897

9998
if hasattr(caps, "supports") and caps.supports:

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ requires-python = ">=3.11"
88
authors = [
99
{ name = "Microsoft MADE:Explorations Team" },
1010
]
11-
keywords = ["amplifier", "copilot", "llm", "ai", "claude"]
11+
keywords = ["amplifier", "copilot", "github", "llm", "ai", "provider"]
1212
classifiers = [
1313
"Development Status :: 3 - Alpha",
1414
"Intended Audience :: Developers",

0 commit comments

Comments
 (0)