Skip to content

Conversation

@MajoorWaldi
Copy link

Majoor Assets Manager is a custom node extension designed to browse, inspect, and organize your generated images, videos, and audio files without leaving the ComfyUI interface.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 38706 to 38708
"description": "An advanced Assets manager for ComfyUI outputs with gallery, metadata inspection, ratings, and tags.",
"tags": ["Assets manager", "gallery", "metadata", "workflow", "utility"]
},

Choose a reason for hiding this comment

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

P1 Badge Remove trailing comma from final custom node entry

The new Majoor Assets Manager block ends with a trailing comma (},) even though it is the final element before the closing array bracket, leaving custom-node-list.json invalid JSON. Running jq empty custom-node-list.json now fails with Expected another array element at line 38709, column 5, so any consumer that parses this list will error out until the comma is removed.

Useful? React with 👍 / 👎.

@MajoorWaldi
Copy link
Author

tested locally with DB: Local, installation and usage OK. Ready to merge

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.

2 participants