Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
66 commits
Select commit Hold shift + click to select a range
c34616f
Add code style guidelines to copilot instructions
gunzip Sep 10, 2025
4a68ce9
OPEX dashboard refactor skeleton
gunzip Sep 10, 2025
7265adf
Implement Azure Dashboard and Alerts Builders with CLI support
gunzip Sep 10, 2025
815601c
Add unit tests for CLI commands, endpoint parsing, Kusto query genera…
gunzip Sep 10, 2025
88e5d23
Add Vitest configuration for testing in opex-dashboard-ts package
gunzip Sep 10, 2025
d00c4e2
Remove build script and update TypeScript configuration to use Vitest…
gunzip Sep 10, 2025
ce88436
Add Zod validation for configuration and update CLI command to use it
gunzip Sep 10, 2025
b8daa9b
Enhance configuration validation with detailed error messages and upd…
gunzip Sep 10, 2025
7932e14
Refactor configuration handling by replacing openapi types with valid…
gunzip Sep 10, 2025
c4c6e78
Refactor OpenAPI type handling by moving OpenAPISpec and type guards …
gunzip Sep 10, 2025
ec20851
Remove comprehensive test, demo CLI, and validation scripts to stream…
gunzip Sep 10, 2025
6408724
Refactor AzureAlertsConstruct and AzureDashboardConstruct to remove r…
gunzip Sep 10, 2025
46817c6
Refactor CLI command to remove template-name option and update tests;…
gunzip Sep 10, 2025
6c416ba
Refactor dashboard generation to use generateDashboard function; stre…
gunzip Sep 10, 2025
ad5fce6
chore: add tsup configuration for building the Opex Dashboard TypeScr…
gunzip Sep 10, 2025
bac876a
Format files
gunzip Sep 10, 2025
726075f
chore: update package.json and add ESLint configuration; refactor imp…
gunzip Sep 10, 2025
c51a964
refactor: update README and CLI commands; replace ts-node with tsx, r…
gunzip Sep 10, 2025
6cf7dc2
refactor: consolidate Azure dashboard and alerts creation in the same…
gunzip Sep 10, 2025
5989eb4
refactor: rename AzureDashboardStack to AzureOpexStack and update rel…
gunzip Sep 10, 2025
e98460d
refactor: update package.json exports and improve TypeScript definiti…
gunzip Sep 10, 2025
c90060b
refactor: update VSCode settings for TypeScript formatting; add depen…
gunzip Sep 10, 2025
22d6d99
refactor: rename AzureOpexConstruct to AzureOpexStack and update rela…
gunzip Sep 10, 2025
693107a
feat: Add Azure Opex Dashboard configuration and implementation
gunzip Sep 10, 2025
2433137
refactor: update .gitignore and clean up package.json dependencies
gunzip Sep 10, 2025
977ed08
fix: replace hardcoded resource group name with config value and upda…
gunzip Sep 10, 2025
180d490
feat: refactor to hexagonal architecture
gunzip Sep 10, 2025
8d1a40c
feat: implement OA3Resolver and Endpoint parsing utilities with Zod v…
gunzip Sep 10, 2025
b982d9d
fix: update CLI entry path in tsup configuration
gunzip Sep 10, 2025
304a56f
feat: implement CreateDashboardStackUseCase and TerraformStackGenerat…
gunzip Sep 10, 2025
7385d31
refactor: reorganize backend configuration and improve error handling…
gunzip Sep 10, 2025
c4267d1
feat: implement addOpexStack function to generate Azure dashboard and…
gunzip Sep 10, 2025
e0c7d10
fix: correct import and usage of addOpexStack in opex.ts
gunzip Sep 10, 2025
421a4b5
Implement code changes to enhance functionality and improve performance
gunzip Sep 11, 2025
1e96665
feat: add contributing guidelines and update example configurations f…
gunzip Sep 11, 2025
c6f5bf9
feat: enable HCL output for Terraform CDKTF code generation
gunzip Sep 11, 2025
d4affba
feat: enhance Kusto query generation and improve alert descriptions
gunzip Sep 11, 2025
ce42111
fix: remove unused generate script from package.json
gunzip Sep 11, 2025
0bd0cef
feat: implement OpenAPI spec resolution with host extraction and enha…
gunzip Sep 11, 2025
19fe7fc
feat: enhance Azure alerts with tenant-specific dashboard URLs and cl…
gunzip Sep 11, 2025
85c6193
refactor: remove commented-out Terraform logic from buildAlertName me…
gunzip Sep 11, 2025
9a9c94f
refactor: simplify dashboard properties template by removing unnecess…
gunzip Sep 11, 2025
c9a768d
feat: implement panel model and factory for dynamic dashboard generation
gunzip Sep 11, 2025
fd0c17a
test: add snapshot-style tests for dashboard properties and Kusto que…
gunzip Sep 11, 2025
b5c01ae
refactor: streamline query construction for response codes in PanelFa…
gunzip Sep 11, 2025
9c6f58a
feat: enhance dashboard properties and query generation with path nor…
gunzip Sep 11, 2025
bdf9a43
refactor: fix indentation and normalize subtitle placeholders in dash…
gunzip Sep 11, 2025
3a6453e
refactor: update Kusto queries to use extend for watermark and simpli…
gunzip Sep 11, 2025
f869c87
refactor: improve alert description formatting and enhance slug gener…
gunzip Sep 11, 2025
c3508a1
feat: add Terraform configuration for Azure portal dashboard and alerts
gunzip Sep 11, 2025
f0b7312
refactor: replace merge function with Zod schema validation for endpo…
gunzip Sep 11, 2025
75240b0
refactor: use nullish coalescing operator for threshold defaults in K…
gunzip Sep 11, 2025
0cc2497
refactor: simplify DashboardConfig defaults and remove mergeConfigWit…
gunzip Sep 11, 2025
05f562c
refactor: update endpoint properties to use snake_case naming convention
gunzip Sep 12, 2025
88c3a77
fix: add missing host entry in azure_dashboard_overrides_config.yaml
gunzip Sep 12, 2025
4bd0120
refactor: replace DashboardConfig with ValidDashboardConfig across th…
gunzip Sep 12, 2025
dc1b3fc
feat: add Infra OPEX Code Review workflow and update related configur…
gunzip Sep 12, 2025
8a95151
feat: add Terraform plan workflow for Infra OPEX and commit generated…
gunzip Sep 12, 2025
15540ef
feat: update Infra OPEX workflows to use artifact upload and download…
gunzip Sep 12, 2025
ee31ace
feat: enable Corepack in Infra OPEX Code Review workflow
gunzip Sep 12, 2025
236e164
feat: add build step for dependencies in Infra OPEX Code Review workflow
gunzip Sep 12, 2025
5782be8
feat: add build step for generating dependencies in Infra OPEX workflow
gunzip Sep 12, 2025
c29d847
fix: remove incorrect build step for dependencies in Infra OPEX Code …
gunzip Sep 12, 2025
6f81d6f
fix: update standalone:copy script to use -p flag for mkdir
gunzip Sep 13, 2025
aa6a084
fix: remove unnecessary generate step from build script and adjust de…
gunzip Sep 13, 2025
34061c5
fix: remove duplicate checkout step in infra_plan_opex workflow
gunzip Sep 14, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions .github/copilot-instructions.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
# Code style

- Do not add jsdoc comments (/** ... */); TypeScript typing is sufficient.
- Use C block syntax (/* ... */) for method-level comments.
- Use C++ style comments (// ...) for inline comments (within method body).
- Nullable or boolean arguments are code smells; prefer configuration objects as arguments.
- Avoid casts and check if a cast is really necessary; do not use unnecessary casts.
- Always remove legacy and unreachable code.
- Avoid creating overly complex or large methods/modules; split into smaller, manageable functions with clear naming.
- Comment unclear code sections with C block comments explaining the reason for the code and, when applicable, the input and output produced.
264 changes: 0 additions & 264 deletions .github/workflows/code_review_infra_exp.yaml

This file was deleted.

67 changes: 67 additions & 0 deletions .github/workflows/code_review_infra_opex.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,67 @@
name: Infra OPEX Code Review

on:
pull_request:
types:
- opened
- synchronize
- reopened
- ready_for_review
paths:
# Trigger the workflow when source files that generate terraform are modified
- "test-opex-api/**/opex.ts"
- "test-opex-api/**/openapi/*.yaml"
# - "**/openapi*.yaml"
# - "**/openapi/**/*.yaml"
# Trigger the workflow when the involved workflows are modified
- ".github/workflows/code_review_infra_opex.yaml"

permissions:
contents: read
pull-requests: write
id-token: write

jobs:
generate_infra:
runs-on: ubuntu-latest
name: Generate Infra OPEX
steps:
- name: Checkout repository
uses: actions/checkout@v3
with:
fetch-depth: 0

- name: Corepack Enable
run: corepack enable

- name: Set up Node.js
uses: actions/setup-node@v3
with:
node-version: "20"
cache: "yarn"

- name: Install dependencies
run: yarn install

- name: Build dependencies
run: yarn build

- name: Generate infra for test-opex-api
run: yarn run generate:infra --workspace=test-opex-api

- name: Upload generated infra artifacts
uses: actions/upload-artifact@v4
with:
name: generated-infra-opex
path: apps/test-opex-api/stacks/

code_review_dev:
needs: generate_infra
uses: pagopa/dx-playground/.github/workflows/infra_plan_opex.yaml@chores/opex-refactor-hex
name: Code Review Infra OPEX Plan
secrets: inherit
with:
environment: opex-dashboard
override_github_environment: "infra-dev"
base_path: test-opex-api/stacks
artifact_name: generated-infra-opex
Loading
Loading