[Request] Check/CLI - Add optional media information input #866

New Issue

claunia · 2026-01-29T16:23:34Z

claunia commented

2026-01-29 16:23:34 +00:00

Originally created by @mnadareski on GitHub (Oct 18, 2025).

Originally assigned to: @mnadareski on GitHub.

Add a new flag that applies to both Check and CLI that allows users to fill in media information. This would likely be a default enable if the user was already in interactive mode and a separate flag-enabled feature otherwise.

Considerations:

There are an extremely large number of fields and having every possible one represented may be difficult
A distinct subset of information, such as basic title, disc number/letter, comments, contents, and ringcodes may be a good place to start
Would any of the information be reasonable to be user input flags? Things like game and disc title could be their own, separate input flags as well as being part of the media input mode.
How many flags is too many?

Originally created by @mnadareski on GitHub (Oct 18, 2025). Originally assigned to: @mnadareski on GitHub. Add a new flag that applies to both Check and CLI that allows users to fill in media information. This would likely be a default enable if the user was already in interactive mode and a separate flag-enabled feature otherwise. Considerations: - There are an extremely large number of fields and having every possible one represented may be difficult - A distinct subset of information, such as basic title, disc number/letter, comments, contents, and ringcodes may be a good place to start - Would any of the information be reasonable to be user input flags? Things like game and disc title could be their own, separate input flags as well as being part of the media input mode. - How many flags is too many?

claunia added the enhancement label 2026-01-29 16:23:34 +00:00

claunia commented

2026-01-29 16:23:35 +00:00

@JohnVeness commented on GitHub (Oct 18, 2025):

I've been using the MPF.Check --load-seed option for adding media information (usually ringcodes, barcodes, comments, and contents, as I'm usually doing verifications so the title etc. pulled from redump is usually OK), and that works very well. I think it would be generally easier to add ringcodes etc. by editing a seed file rather than trying to fill them in as individual command-line options, so I'm not sure that separate flags would be a good idea.

I don't generally use interactive mode, but I can see that it would be useful to enter media information there, as the seed file thing can be a little complicated.

@JohnVeness commented on GitHub (Oct 18, 2025): I've been using the MPF.Check `--load-seed` option for adding media information (usually ringcodes, barcodes, comments, and contents, as I'm usually doing verifications so the title etc. pulled from redump is usually OK), and that works very well. I think it would be generally easier to add ringcodes etc. by editing a seed file rather than trying to fill them in as individual command-line options, so I'm not sure that separate flags would be a good idea. I don't generally use interactive mode, but I can see that it would be useful to enter media information there, as the seed file thing can be a little complicated.

claunia commented

2026-01-29 16:23:35 +00:00

@mnadareski commented on GitHub (Oct 18, 2025):

Seed files are definitely a way of doing it, but it relies heavily on having a submission info JSON template available or passed in from some external program. I'm also surprised to hear that anyone is using the seed file input and that it works :D This is more of an overall consideration of how to handle user input in a less power-user way.

@mnadareski commented on GitHub (Oct 18, 2025): Seed files are definitely a way of doing it, but it relies heavily on having a submission info JSON template available or passed in from some external program. I'm also surprised to hear that anyone is using the seed file input and that it works :D This is more of an overall consideration of how to handle user input in a less power-user way.

claunia commented

2026-01-29 16:23:36 +00:00

@JohnVeness commented on GitHub (Oct 18, 2025):

Yes, it seems to work perfectly :). I agree it is a bit power-user-y though.

@JohnVeness commented on GitHub (Oct 18, 2025): Yes, it seems to work perfectly :). I agree it is a bit power-user-y though.

claunia commented

2026-01-29 16:23:36 +00:00

@Deterous commented on GitHub (Oct 19, 2025):

Seed works for power users who want all info filled in
Flags suck, there's already many of them. EIther output the template to be filled in later (status quo), or ask for info in interactive mode (only when called in interactive mode)
Interactive mode for all details would be annoying as half the stuff is blank/not valid. Title and ringcodes are the only things that are always needed. Disc Title/Num are only needed for multi-disc releases. Foreign title and barcodes are sometimes relevant. Edition and category are redump-specific not useful for MPF in general, and are usually just Games/Original. Whether any of these things are needed cannot be detected via MPF, so having a bunch of questions to skip through would be a terrible user experience.
Contents (and Comments?) are usually filled in after inspecting the dump, so would be weird to have question for it at dump-time.

@Deterous commented on GitHub (Oct 19, 2025): - Seed works for power users who want all info filled in - Flags suck, there's already many of them. EIther output the template to be filled in later (status quo), or ask for info in interactive mode (only when called in interactive mode) - Interactive mode for all details would be annoying as half the stuff is blank/not valid. Title and ringcodes are the only things that are always needed. Disc Title/Num are only needed for multi-disc releases. Foreign title and barcodes are sometimes relevant. Edition and category are redump-specific not useful for MPF in general, and are usually just Games/Original. Whether any of these things are needed cannot be detected via MPF, so having a bunch of questions to skip through would be a terrible user experience. - Contents (and Comments?) are usually filled in after inspecting the dump, so would be weird to have question for it at dump-time.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: SabreTools/MPF#866