release v0.1.18#1
Conversation
|
DescriptionThis PR introduces the Type of Change
How Has This Been Tested?
Screenshots (if applicable)N/A Checklist
Related IssuesN/A Summary By MatterAI
🔄 What ChangedThis pull request deprecates the standalone 🔍 Impact of the ChangeThis is a significant architectural refactoring that consolidates Matter AI components into a single enterprise Helm chart, simplifying deployment and management. It introduces a new 📁 Total Files Changed
🧪 Test AddedN/A 🔒Security VulnerabilitiesN/A Tip Quality Recommendations
Tanka Poem ♫
Sequence DiagramsequenceDiagram
participant Dev as Developer
participant Helm as Helm Chart
participant K8sAPI as Kubernetes API
participant MCP as Matter AI MCP Service
participant Backend as Matter AI Backend Service
participant Ingress as Kubernetes Ingress
participant Client as External Client
Dev->>Helm: helm install/upgrade (gravity-matter-enterprise)
Helm->>K8sAPI: Deploy ConfigMap (matter-ai-mcp-config)
Helm->>K8sAPI: Deploy Deployment (matter-mcp-server)
Helm->>K8sAPI: Deploy Service (matter-mcp-server-service)
Helm->>K8sAPI: Deploy Ingress (matter-mcp-server-ingress)
K8sAPI-->>Helm: Resources created
Helm-->>Dev: Deployment successful
activate MCP
MCP->>K8sAPI: Read ConfigMap (matter-ai-mcp-config)
Note over MCP: MATTER_API_ENDPOINT set to matterBackend service
Client->>Ingress: HTTP/S Request (matterai-mcp.example.com)
Ingress->>MCP: Route request
MCP->>Backend: Call matterBackend API (http://matter-backend-service.namespace.svc.cluster.local:port)
Backend-->>MCP: API Response
MCP-->>Ingress: Response to client
Ingress-->>Client: Response
deactivate MCP
|
![matterai-app[bot]](https://avatars.githubusercontent.com/in/474549?s=60&v=4){kind=small}
| description: Helm chart to deploy Matter AI Code Review Backend for Enterprise | ||
| type: application | ||
| icon: https://matterai.dev/favicon.png | ||
| appVersion: 0.1.16 | ||
| version: 0.1.17 | ||
| appVersion: 0.1.58 | ||
| version: 0.1.18 | ||
| maintainers: | ||
| - name: Vatsal Bajpai | ||
| email: vatsal@matterai.dev | ||
| email: vatsal@matterai.so |
There was a problem hiding this comment.
📝 Documentation Gap
Issue: Email domain changed from matterai.dev to matterai.so without updating description
Fix: Update description to reflect new domain or maintain consistency
Impact: Ensures documentation accuracy and brand consistency
| description: Helm chart to deploy Matter AI Code Review Backend for Enterprise | |
| type: application | |
| icon: https://matterai.dev/favicon.png | |
| appVersion: 0.1.16 | |
| version: 0.1.17 | |
| appVersion: 0.1.58 | |
| version: 0.1.18 | |
| maintainers: | |
| - name: Vatsal Bajpai | |
| email: vatsal@matterai.dev | |
| email: vatsal@matterai.so | |
| description: Helm chart to deploy Matter AI Code Review Backend for Enterprise | |
| type: application | |
| icon: https://matterai.so/favicon.png | |
| appVersion: 0.1.58 | |
| version: 0.1.18 | |
| maintainers: | |
| - name: Vatsal Bajpai | |
| email: vatsal@matterai.so |
| {{- range $key, $value := $component.deployment.env }} | ||
| - name: {{ $key }} | ||
| {{- if eq $key "MATTER_API_ENDPOINT" }} | ||
| {{- if eq $componentName "matterMCP" }} | ||
| value: "http://{{ $.Values.components.matterBackend.service.name }}.{{ $.Values.global.namespace }}.svc.cluster.local:{{ $.Values.components.matterBackend.service.port }}" | ||
| {{- else }} | ||
| {{- if kindIs "map" $value }} | ||
| {{- toYaml $value | nindent 14 }} | ||
| {{- else }} | ||
| value: {{ $value | quote }} | ||
| {{- end }} | ||
| {{- end }} | ||
| {{- else }} | ||
| {{- if kindIs "map" $value }} | ||
| {{- toYaml $value | nindent 14 }} | ||
| {{- else }} | ||
| value: {{ $value | quote }} | ||
| {{- end }} | ||
| {{- end }} | ||
| {{- end }} |
There was a problem hiding this comment.
🛠️ Code Refactor
Issue: Complex nested conditional logic for MATTER_API_ENDPOINT makes template hard to read and maintain
Fix: Simplify the conditional structure and improve readability
Impact: Reduces template complexity and improves maintainability
| {{- range $key, $value := $component.deployment.env }} | |
| - name: {{ $key }} | |
| {{- if eq $key "MATTER_API_ENDPOINT" }} | |
| {{- if eq $componentName "matterMCP" }} | |
| value: "http://{{ $.Values.components.matterBackend.service.name }}.{{ $.Values.global.namespace }}.svc.cluster.local:{{ $.Values.components.matterBackend.service.port }}" | |
| {{- else }} | |
| {{- if kindIs "map" $value }} | |
| {{- toYaml $value | nindent 14 }} | |
| {{- else }} | |
| value: {{ $value | quote }} | |
| {{- end }} | |
| {{- end }} | |
| {{- else }} | |
| {{- if kindIs "map" $value }} | |
| {{- toYaml $value | nindent 14 }} | |
| {{- else }} | |
| value: {{ $value | quote }} | |
| {{- end }} | |
| {{- end }} | |
| {{- end }} | |
| env: | |
| {{- range $key, $value := $component.deployment.env }} | |
| - name: {{ $key }} | |
| {{- if and (eq $key "MATTER_API_ENDPOINT") (eq $componentName "matterMCP") }} | |
| value: "http://{{ $.Values.components.matterBackend.service.name }}.{{ $.Values.global.namespace }}.svc.cluster.local:{{ $.Values.components.matterBackend.service.port }}" | |
| {{- else if kindIs "map" $value }} | |
| {{- toYaml $value | nindent 14 }} | |
| {{- else }} | |
| value: {{ $value | quote }} | |
| {{- end }} | |
| {{- end }} |
| env: | ||
| MATTER_API_ENDPOINT: "example.com" |
There was a problem hiding this comment.
🛠️ Code Refactor
Issue: Hardcoded example.com domain in MATTER_API_ENDPOINT environment variable
Fix: Use a more descriptive placeholder or template variable
Impact: Improves configuration clarity and reduces confusion during deployment
| env: | |
| MATTER_API_ENDPOINT: "example.com" | |
| env: | |
| MATTER_API_ENDPOINT: "http://matter-backend.example.com" |
|
/matter release-notes |
What's ChangedSummaryThis release marks a significant architectural shift by deprecating the standalone New Features
Bug fixesSkip this section if none. Commits
Dependencies added
|
1 participant
Description
Type of Change
How Has This Been Tested?
Screenshots (if applicable)
Checklist
Related Issues