Repository navigation
Release 6.2.18: report specs by registered name and the pyegeria.report_specs entry points (ISSUE-173) - #523
Merged
Conversation
… deprecated (ISSUE-173) A dotted analytic_function, or metric_trend's metric_path, still resolves under the ISSUE-169 checks but now raises a DeprecationWarning and logs once, naming the registered name to use. metric_path previously imported any caller-supplied dotted path with no allowlist check; it now goes through the same resolver. pyegeria's own specs (the SoA summary and the analytic demos) now use registered names. Signed-off-by: Dan Wolfson <dan.wolfson@pdr-associates.com>
… group; deprecate the env-var sources (ISSUE-173) refresh_report_specs() reads the pyegeria.report_specs group (alias trellis.report_specs) first, each entry point isolated like any other CONFIG source. pyegeria's own analytic demo specs move to that group and leave config.json. config.json's "Pyegeria Report Spec Modules", PYEGERIA_REPORT_SPEC_MODULES and PYEGERIA_REPORT_FORMATS_JSON/_MODULES still load in 6.2.18 with one DeprecationWarning per refresh, and go in 6.2.19. Docs, CLAUDE.md, AGENTS.md and ISSUE-173 updated. Signed-off-by: Dan Wolfson <dan.wolfson@pdr-associates.com>
Signed-off-by: Dan Wolfson <dan.wolfson@pdr-associates.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Release 6.2.18 (ISSUE-173). It has two commits plus the version bump, all on top of 6.2.17 (#522).
Report specs from installed packages.
refresh_report_specs()now reads apyegeria.report_specsentry-point group, and alsotrellis.report_specsas an alias.FormatSetDict(or a dict of label toFormatSet), or names such a dict directly.refresh_report_specs(), and the rest still load.analytic_demos), so the"Pyegeria Report Spec Modules"key is gone fromconfig/config.json.Deprecated, and removed in 6.2.19. These still load in this release:
"Pyegeria Report Spec Modules"PYEGERIA_REPORT_SPEC_MODULESPYEGERIA_REPORT_FORMATS_JSONPYEGERIA_REPORT_FORMATS_MODULESWhen any of them is set, each refresh raises one
DeprecationWarningand logs one WARNING line naming the replacement. If a config entry names the same loader as an installed entry point, it is treated as that entry point rather than as a clash.Registered names in report specs. This applies to a report spec's
analytic_functionand tometric_trend'smetric_path.DeprecationWarningnaming the registered name to use, and logs once per path. 6.2.19 refuses them.metric_pathwas imported with no allowlist check at all; it now goes through the same resolver.Docs updated:
output-formats-and-report-specs.md,CLAUDE.md,AGENTS.md, the view README, and ISSUE-173.Tests: new
tests/micro-tests/test_issue_173_*.py. The full micro-test suite passes on Python 3.13 (1722 collected, 0 failed).Downstream:
report_specs_handler.pyswitches tocount_elementsandmetric_trend.pyegeria.report_specsentry point (already on trellis main, df407a6c).🤖 Generated with Claude Code
https://claude.ai/code/session_015pdMuPkqttD1yKAKJLS9Fc