Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 5, 2025

What

  • establish an Architecture Decision Record (ADR) directory with an initial record adopting the ADR process
  • update contributor guidance to require reviewing and extending ADRs for architectural changes
  • bump the project version to 0.26.58 to reflect the new documentation process

Why

  • capture architectural decisions so future contributors understand design rationale and avoid conflicting changes

Affects

  • repository-wide contributor workflow expectations
  • project metadata versioning

Testing

  • uv run ruff check .
  • uv run pytest

Documentation

  • added docs/adr/0001-adopt-architecture-decision-records.md

https://chatgpt.com/codex/tasks/task_e_68e24a490f8c8328b28eeb6b2635d77b

Copilot AI review requested due to automatic review settings October 5, 2025 10:49
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR establishes an Architecture Decision Record (ADR) process to document design decisions and rationale for future contributors.

  • Introduces an ADR directory with an initial record documenting the adoption of the ADR process
  • Updates contributor guidance to require ADR review and creation for architectural changes
  • Bumps project version to 0.26.58 to reflect the new documentation workflow

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

File Description
pyproject.toml Increments version to 0.26.58
docs/adr/0001-adopt-architecture-decision-records.md Establishes the ADR process and template
docker/pyproject.deps.toml Updates version to match main pyproject.toml
AGENTS.md Adds guidance for contributors to review and create ADRs

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link

github-actions bot commented Oct 5, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex/loader
   __init__.py6282496%201, 256, 258–260, 276, 362–367, 944–949, 1072–1074, 1085–1087, 1099–1101, 1374
mcp_plex/server
   __init__.py6142995%43–44, 119–120, 148, 252, 256, 277–280, 297, 362, 365, 402, 420–421, 458, 1109, 1131–1137, 1173, 1191, 1196, 1214, 1338, 1375
   __main__.py440%3–8
   config.py48785%50, 52–55, 65, 76
TOTAL14966496% 

Tests Skipped Failures Errors Time
100 0 💤 0 ❌ 0 🔥 54.221s ⏱️

@Teagan42 Teagan42 merged commit 126f7d7 into main Oct 5, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/implement-adr-for-architectural-changes branch October 5, 2025 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants