Skip to content

update to cmem-plugin-template v9.5.0 - #5

Merged
seebi merged 3 commits into
mainfrom
feature/updateTemplate
Sep 6, 2026
Merged

update to cmem-plugin-template v9.5.0#5
seebi merged 3 commits into
mainfrom
feature/updateTemplate

Conversation

@seebi

@seebi seebi commented Sep 4, 2026

Copy link
Copy Markdown
Member

Updates this repository to cmem-plugin-template v9.6.1 via copier update, and refreshes the locked dependencies that come with it.

⚠️ There is already an open request touching the template here: #3 (tmp/tryOutPluginBaseRC1). This one supersedes it or should be closed in its favour — please do not merge both.

task check passes locally except integration tests, which need a live Corporate Memory instance (or Docker/service credentials) that was not available where this ran. CI should have it.


Updated to v9.6.1. This request was originally raised against v9.5.0; a second commit moves it to cmem-plugin-template v9.6.1, which restores the [tool.pytest.ini_options] table. Without it pytest's rootdir walk escapes the project directory, so pytest-dotenv stops finding .env and the Corporate Memory integration tests run unconfigured.

Replays the template diff onto this repository and refreshes the locked
dependencies that come with it.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UhUJZ7EwjXm7mN2jAdZzvn
@seebi
seebi requested a review from msaipraneeth September 4, 2026 09:55
@seebi

seebi commented Sep 4, 2026

Copy link
Copy Markdown
Member Author

Reviewer chosen automatically as part of the template-update sweep.

The main author of this plugin is Sebastian Tramp (48% of the commits touching cmem_plugin_*/ on the default branch), but they could not be assigned:

  • Sebastian Tramp (48%) — opened this request and cannot review it.

So review falls to the next-highest contributor who can be assigned here: Saipraneeth Muddana (40% of those commits).

@seebi seebi self-assigned this Sep 4, 2026
@seebi

seebi commented Sep 4, 2026

Copy link
Copy Markdown
Member Author

Taking this one as maintainer. I am the main author of this plugin (48% of the commits touching cmem_plugin_*/ on the default branch), so I will review and merge it myself.

GitHub does not allow requesting a review from the author of a pull request, so this is recorded as an assignment instead of a review request. The automatically chosen reviewer (@msaipraneeth) has been removed from this pull request, since review falls to me.

@seebi
seebi removed the request for review from msaipraneeth September 4, 2026 16:54
seebi and others added 2 commits September 6, 2026 14:17
Replays the v9.5.0 -> v9.6.1 template diff on top of the existing update. This
restores the [tool.pytest.ini_options] table, whose absence let pytest's rootdir
walk escape the project directory.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UhUJZ7EwjXm7mN2jAdZzvn
The project fixture called projects.create_item on a fixed id without clearing
it first. Any run that died before the teardown left validate_entities_test_project
behind on the shared Corporate Memory instance, and every later run then errored
at setup with "Item with id validate_entities_test_project already exists".

Deleting with skip_if_missing before creating matches what cmem-plugin-kafka
already does in tests/utils.make_project.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UhUJZ7EwjXm7mN2jAdZzvn
@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown

Coverage

Coverage Report
File Stmts Miss Cover Missing
init.py 0 0 100%
validate_entities/init.py 0 0 100%
validate_entities/state.py 10 0 100%
validate_entities/task.py 134 38 72% 192 201 212 223-226 231 233-235 249 263-264 286 291 321 323-324 327-330 333-336 338 340-342 344 349 352-353 355 357-358
validate_graph/init.py 0 0 100%
validate_graph/state.py 22 0 100%
validate_graph/task.py 76 6 92% 180-181 188-189 192 199
TOTAL 242 44 82%  

Tests Skipped Failures Errors Time
6 0 💤 0 ❌ 0 🔥 51.823 ⏱

@seebi
seebi merged commit dc47a5e into main Sep 6, 2026
2 checks passed
@seebi
seebi deleted the feature/updateTemplate branch September 6, 2026 18:49
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.

1 participant