Use model-declared ION variable names

[cjc0013]

Summary:

• Read the model input and output variable names from model.json.
• Keep data_in / data_out as fallbacks for older models that omit variable metadata.

Test plan:

• python -m py_compile tools\run.py
• git diff --check

Closes #1

[theblackapple5150]

Thanks for this — this is the right direction and fixes an issue I wanted to address.

I’m going to request a couple small cleanup changes before merging:

• Add the missing comma in the payload dict
• Remove the duplicate meta.json loading block
• Remove the duplicate “Saved as” print
• Adjust output validation so it accepts either the model-defined output variable or data_out as a fallback, instead of requiring both

Once that’s cleaned up, I’m good with merging this.

[cjc0013]

Thanks, fixed those cleanup items in the latest commit.

What changed:

• Added the payload dict comma.
• Kept the single metadata-loading path and single final saved-output print.
• Updated output validation so the response can contain either the model-declared output variable or the legacy data_out fallback.

Verified locally with:

• python -m py_compile tools\run.py
• python -m compileall -q tools ioncli security
• git diff --check