Dev by sir-stevie · Pull Request #2 · Summit-Sailors/EZPZ

sir-stevie · 2025-07-07T23:21:49Z

No description provided.

… more files

… files

github-advanced-security · 2025-07-07T23:22:47Z

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

…d 37 more files

Distortedlogic · 2025-07-17T18:03:02Z

@@ -1,3 +1,4 @@
 [ezpz_pluginz]


lets give user option to use either dedicated ezpz.toml or add [tool.ezpz] to their pyproject.toml

Distortedlogic · 2025-07-17T18:04:18Z

+)
+from ezpz_pluginz.toml_schema import load_config
+
+app = typer.Typer(name="ezplugins", pretty_exceptions_show_locals=False, pretty_exceptions_short=True)


lets name the cli ezpz less chars to type and easier to remember. also notice that there is a theme of using z instead of s ie ezpz_pluginz vs ezpz_plugins

Distortedlogic · 2025-07-17T18:05:38Z

+@app.command(name="help")
+def show_help(command: str = typer.Argument(None, help="Show help for a specific command")) -> None:
+  if command:
+    command_help = {


lets put the cmd help obj in a json file, or just separate py file.

Distortedlogic · 2025-07-17T18:07:10Z

+  logger.info("AVAILABLE COMMANDS:")
+  logger.info("")
+
+  commands = [


i thought typer add builtin "help" functionality via docs strings?

it does have builtin help functionality, I'll adapt to that style

Distortedlogic · 2025-07-17T18:09:11Z

+
+
+@app.command(name="update")
+def update_plugin(


i think push might be more clear than update .. seems this is for pushing a plugin to the registry?

Distortedlogic · 2025-07-17T18:12:26Z

+    logger.info(f"   {plugin.description}")
+
+  logger.info("")
+


i feel like we should put all the registry related stuff in a subcmd .. ie ezpz registry ...

Distortedlogic · 2025-07-17T18:15:35Z

+import logging
+from typing import ClassVar, Optional
+from pathlib import Path
+


we should be using structlog?

Distortedlogic · 2025-07-17T18:17:36Z

+            self.macro_data.append(metadata)
+          except (KeyError, ValueError) as e:
+            logging.getLogger(__name__).warning(f"Failed to create plugin metadata: {e}")
+    super().visit_Call(node)


using super() does not seem correct. the visitor pattern typical handles the recurrsion in the base class from

I'll look deeper into this

I removed the super() call and explicitly controlled recursion by returning False when processing ezpz_plugin_collect calls, ensuring child nodes are not traversed redundantly.

Distortedlogic · 2025-07-17T18:22:14Z

 [tool.rye]
-dev-dependencies = ["painlezz-macroz"]
+dev-dependencies = [
+    "painlezz-macroz",


macroz is only a dev dep? also lets remove painlezz from the name and just call it macroz

The noop.py macros (class_macro, func_macro) are used at runtime by plugin developers when they write @ezpz_plugin_collect decorators

The macro system is integral to how ezpz_pluginz works - it's used by end users writing plugins, not just during development

metadata collection happens during build/analysis time, the decorators themselves are runtime components

there might be a split where:

The macro decorators (noop.py) are runtime dependencies
The metadata collection visitors (macro_metadata_collector.py) are dev/build-time dependencies

It's somehow both runtime and build, I can adapt a hybrid config if possible (will look it up)

Distortedlogic · 2025-07-17T18:25:16Z

+from typing import Any
+
+
+def register_plugin() -> dict[str, Any]:


we should use something with narrower typing than dict[str, Any] for users to register a plugin. most likly a pydantic model so that we can also run validations at the same time via the model

…e files

…re files

sir-stevie and others added 21 commits June 13, 2025 23:33

Update Cargo.toml, Cargo.toml, ezpz.toml, and 41 more files

2601e12

Update Cargo.toml, ezpz.toml, pyproject.toml, and 17 more files

4241832

Update README.md

6c5d9bf

Update mod.rs, mod.rs, mod.rs, and 2 more files

af57e1f

Update .gitignore, build.rs, Cargo.toml, and 12 more files

e225a89

Update Cargo.toml, ezpz.toml, pyproject.toml, and 6 more files

35eb31e

Update settings.json, ezpz_rust_ti.py, pyproject.toml, and 19 more files

37c4a0f

Update settings.json, Cargo.toml, mod.rs, and 5 more files

62cd615

Update settings.json, clippy.toml, ezpz_rust_ti.py, and 6 more files

74f3124

Update pyproject.toml, __init__.py, __cli__.py, and 1 more file

dd5c792

Update __cli__.py and registry.py

2360239

Update .dockerignore, docker-compose.yml, justfile, and 25 more files

3e86c89

Update Dockerfile, init.sql, justfile, and 20 more files

36a52b7

Update Cargo.toml, __init__.py, noop.py, and 154 more files

c8e8b7e

Update __cli__.py, e_polars_namespace.py, registry.py, and 13 more files

616d90d

Update main-ci-cd.yml, plugin-ci-cd.yml, publish-packages.yml, and 14…

772d98b

… more files

Update audit.yml, core.yml, main-ci-cd.yml, and 23 more files

6a9df85

Update .gitignore, pyproject.toml, pyproject.toml, and 2 more files

84422b9

Updates

85cd050

Update audit.yml, core.yml, plugins.yml, and 2 more files

d7ebf58

Update dependabot.yml, build-rust.nu, generate-report.nu, and 31 more…

99fa4d5

… files

sir-stevie and others added 6 commits July 12, 2025 16:53

Update Cargo.toml, Cargo.toml, ezpz_rust_ti.py, and 9 more files

daed0e6

backup before cleanup

8615fbb

Update .rustfmt.toml, Cargo.toml, Cargo.toml, and 34 more files

13efd07

Merge branch 'dev' of github.com:Summit-Sailors/EZPZ into dev

fd811bb

Update analyze_plugins.py, register_plugins.py, update_plugins.py, an…

9594ee8

…d 37 more files

Update audit.yml, test_macroz.py, __cli__.py, and 1 more file

b911659

sir-stevie requested a review from Distortedlogic July 14, 2025 20:45

Update audit.yml, plugins.yml, actions.just, and 6 more files

d42ec0a

Update __cli__.py, models.py, local.py, and 6 more files

be3caa2

Distortedlogic approved these changes Jul 17, 2025

View reviewed changes

sir-stevie added 5 commits July 18, 2025 18:33

Update analyze_plugins.py, build-plugin.nu, build-rust.nu, and 47 mor…

b21736f

…e files

Update convert_sarif.py, audit.yml, codeql.yml, and 3 more files

d078475

Update generate-report.nu, plugin_manager.py, plugin_ops.nu, and 3 mo…

5039074

…re files

Update README.md

b9cefb3

Update core.yml and requirements-dev.lock

9981551

sir-stevie requested a review from Distortedlogic July 23, 2025 20:24

Update Cargo.toml, __init__.py, lockfile.py, and 16 more files

6f71d76

Distortedlogic merged commit b9893e6 into main Aug 16, 2025
13 of 16 checks passed

		from typing import Any


		def register_plugin() -> dict[str, Any]:

Conversation

sir-stevie commented Jul 7, 2025

Uh oh!

github-advanced-security AI commented Jul 7, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants