Skip to content

chore: add .github/copilot-instructions.md#80

Merged
diberry merged 8 commits intoAzure-Samples:mainfrom
diberry:chore/copilot-instructions
May 8, 2026
Merged

chore: add .github/copilot-instructions.md#80
diberry merged 8 commits intoAzure-Samples:mainfrom
diberry:chore/copilot-instructions

Conversation

@diberry
Copy link
Copy Markdown
Collaborator

@diberry diberry commented May 8, 2026

Adds project-level Copilot instructions covering DocumentDB sample conventions: environment variables, vector index patterns, collection lifecycle, and language-specific notes.

Extracted from PR #74 to keep that PR focused on Article 2 select-algorithm samples.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
diberry added a commit to diberry/documentdb-samples that referenced this pull request May 8, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
diberry and others added 7 commits May 8, 2026 09:54
…, and code patterns

- Add OpenAI SDK dependencies for Go, Java, and .NET
- Add python-dotenv and godotenv dependencies
- Fix DATA_FILE_WITH_VECTORS default from ./Hotels_Vector.json to ../data/Hotels_Vector.json
- Add AZURE_OPENAI_EMBEDDING_API_VERSION and MONGO_CONNECTION_STRING to env var table
- Add Authentication section documenting passwordless (OIDC) and connection string auth
- Add Sample Execution Pattern section with consistent lifecycle, naming conventions,
  standard search query, and vector search pipeline structure
- Add Repository Structure overview
- Clarify Rule 1 exceptions (mongocluster.cosmos.azure.com, cosmosSearch, VS Code extension)
- Fix Rule 3 to reference env var with shared data path default
- Fix Rule 6 to list actual .NET config sections
- Add rules for COS similarity, output files, and index type availability

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Document the agent sample patterns as a separate convention section covering:
- Planner/Synthesizer two-agent architecture with 3 LLM deployments
- Agent-specific env vars vs quickstart env var mapping
- Entry point pattern (upload/agent/cleanup)
- Language-specific SDK stacks (Go raw vs TS LangChain)
- OIDC authentication scopes
- IVF numLists discrepancy (10 vs 1)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Remove python-dotenv and godotenv from dependency lists
- Add Rule 10: no dotenv libraries
- Add 'Running Samples' section with per-language CLI examples
- Include agent sample multi-LLM invocation example
- Add Windows PowerShell note

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Main file trimmed from 377 to 156 lines (core rules, env vars, patterns)
- .github/instructions/cli-examples.instructions.md: Bash/PowerShell invocation
  examples, scoped to ai/** paths
- .github/instructions/agent-samples.instructions.md: Multi-LLM agent convention,
  scoped to ai/vector-search-agent-*/** paths

Follows GitHub Copilot guidance to keep copilot-instructions.md concise and use
path-specific .instructions.md files for detailed/scoped content.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- copilot-instructions.md now 107 lines (from 377 original)
- New: .github/instructions/execution-patterns.instructions.md
  Scoped to ai/vector-search-*/** — covers auth modes, lifecycle,
  naming conventions, search query, pipeline structure

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Add Sample Categories section (quickstart vs agent distinction)
- Add IVF numLists footnote (quickstart=1, agents=10 intentional)
- Clarify .NET appsettings.json + env var override pattern
- Add Rule 11: collection naming convention (hotels_{algorithm})
- Add Rule 12: k=5 for vector search results
- Make DescriptionVector explicit in pipeline template
- Add note that CLI examples apply to all 3 algorithms

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@diberry diberry merged commit c58bac6 into Azure-Samples:main May 8, 2026
1 check passed
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