Skip to content

feat: KM Modularity: Enhance documentation, code cleanup, and infrastructure updates#1016

Merged
Avijit-Microsoft merged 241 commits into
mainfrom
dev
Jul 24, 2026
Merged

feat: KM Modularity: Enhance documentation, code cleanup, and infrastructure updates#1016
Avijit-Microsoft merged 241 commits into
mainfrom
dev

Conversation

@Pavan-Microsoft

Copy link
Copy Markdown
Contributor

Purpose

This pull request introduces several significant changes focused on project cleanup, CI/CD improvements, and renaming for clarity and alignment. The most important updates include adding a release automation workflow, updating workflow and script references from "CKM" to "KM", restructuring and cleaning up infrastructure and script files, and improving .dockerignore to reduce build context size.

CI/CD and Release Automation:

  • Added a new GitHub Actions workflow .github/workflows/create-release.yml to automate semantic versioning and GitHub releases using semantic-release.

Naming and Reference Updates:

  • Updated all workflow and script references from "CKM"/"CKMv2" to "KM"/"KM V1" for consistency, including job names, variables, resource group prefixes, and notification content in .github/workflows/bicep_deploy.yml, .github/workflows/job-send-notifications.yml, and .github/workflows/validate-bicep-params.yml. [1] [2] [3] [4] [5] [6] [7]

Infrastructure and Script Organization:

  • Updated script paths in .devcontainer/setup_env.sh and workflow steps to use the new infra/scripts/pre-provision/ directory and corresponding KM V1 scripts. [1] [2]
  • Increased default quota parameters for GPT_MIN_CAPACITY and added TEXT_EMBEDDING_MIN_CAPACITY in the deployment workflow.

Build and Development Environment Cleanup:

  • Improved .dockerignore to exclude virtual environments, build artifacts, source directories, and CI/CD config folders, reducing Docker build context and improving security.
  • Removed .coveragerc omit section for test files, possibly to include all files in coverage analysis.
  • Updated .devcontainer/devcontainer.json to add PowerShell and remove unnecessary Azure CLI Bicep options.

Documentation and Metadata Updates:

  • Enhanced the README.md introduction for clarity and focus on AI-driven analysis use cases.
  • Removed CODE_OF_CONDUCT.md and CONTRIBUTING.md files, possibly to consolidate or update contribution guidelines elsewhere. [1] [2]
  • Removed .gitattributes, which may affect line ending and merge behavior.

Does this introduce a breaking change?

  • Yes
  • No

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

Comment thread src/api/modules/ingestion/azure_storage.py Fixed
Comment thread src/api/modules/ingestion/azure_storage.py Fixed
Comment thread infra/scripts/post-provision/connect-data.py Fixed
Comment thread infra/scripts/post-provision/connect-data.py Fixed
Comment thread src/api/modules/document_intelligence/enrichment_agent.py Fixed
Comment thread src/api/modules/ingestion/queue_worker.py Fixed
Comment thread src/api/modules/runtime/registry.py Fixed
Comment thread src/api/modules/ingestion/router.py Fixed
Comment thread src/api/modules/ingestion/router.py Fixed
Comment thread src/api/modules/rag/service.py Fixed
Copilot AI review requested due to automatic review settings July 24, 2026 09:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

Comment thread src/api/main.py
Comment thread src/api/modules/insights/router.py
Copilot AI review requested due to automatic review settings July 24, 2026 10:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

Comment thread src/api/modules/data_sources/base.py
Comment thread src/api/modules/data_sources/base.py
Comment thread src/api/modules/data_sources/base.py
Comment thread src/api/modules/data_sources/base.py
Comment thread src/api/modules/data_sources/base.py
Comment thread src/api/modules/data_sources/base.py
Comment thread src/api/modules/data_sources/base.py
Comment thread src/api/storage/base.py
Comment thread src/api/storage/base.py
Comment thread src/api/storage/base.py
@Pavan-Microsoft
Pavan-Microsoft marked this pull request as draft July 24, 2026 11:09
@Pavan-Microsoft
Pavan-Microsoft marked this pull request as ready for review July 24, 2026 11:10
@Avijit-Microsoft
Avijit-Microsoft merged commit 1d31d12 into main Jul 24, 2026
12 checks 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.