Skip to content

Planning codebase modernization and reliability improvements - #8

Draft
lassault with Copilot wants to merge 3 commits into
masterfrom
copilot/modernize-codebase-and-add-tests
Draft

Planning codebase modernization and reliability improvements#8
lassault with Copilot wants to merge 3 commits into
masterfrom
copilot/modernize-codebase-and-add-tests

Conversation

Copilot AI commented May 15, 2026

Copy link
Copy Markdown
  • Baseline reviewed (legacy src layout, runtime flow, docs, CI)
  • Phase 1 – Project packaging & tooling
    • Introduced pyproject.toml as authoritative project config
    • Added executable package entrypoint (osniffy)
    • Refactored parsing, DB, dashboard, reader/sniffer, and CLI into osniffy/
  • Phase 2 – Reliability test suite
    • Added deterministic parser/frame tests
    • Added config/CLI tests for success and failure paths
    • Added DB repository behavior tests (filtering, row mapping, retries, range)
    • Added reader and dashboard tests
    • Achieved coverage gate (>70%)
  • Phase 3 – CI/CD and provisioning modernization
    • Added CI workflow skeleton (lint/type/tests/build/security)
    • Added tag-based release workflow
    • Added declarative Grafana provisioning assets (datasource + dashboard providers)
    • Aligned CI dependency audit with modern packaging
  • Phase 4 – Security, docs, and operational hardening
    • Added root-level .env.example with secure guidance
    • Refreshed README.md with architecture, supported platforms, and as-is spec
    • Added CONTRIBUTING.md and CHANGELOG.md
    • Removed accidental artifacts and ignored them (.coverage)
    • Tightened workflow token permissions and fixed CodeQL findings
  • Final validation
    • ruff check osniffy/ tests/
    • mypy osniffy/
    • pytest (126 passed, coverage > 90%)
    • codeql_checker clean (0 alerts)

Copilot AI and others added 2 commits May 15, 2026 19:26
Agent-Logs-Url: https://github.com/lassault/OSniffy/sessions/238417f7-09ca-4525-82dd-0f8d4b689013

Co-authored-by: lassault <40122711+lassault@users.noreply.github.com>
Comment thread .github/workflows/ci.yml Fixed
Comment thread .github/workflows/ci.yml Fixed
Comment thread .github/workflows/ci.yml Fixed
Comment thread .github/workflows/ci.yml Fixed
Comment thread osniffy/sniffer.py Fixed
Copilot AI requested a review from lassault May 18, 2026 16:39
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.

3 participants