fix: move argument validation before sys.argv access in mkconcore.py (fixes #267)

[GaneshPatil7517]

Hey pradeeban sir,

Fixes #267.

mkconcore.py crashes with an IndexError when it is run without arguments. This happens because sys.argv[1], sys.argv[2], and sys.argv[3] are accessed before checking how many arguments were provided.

To fix this, the existing len(sys.argv) < 4 validation is moved earlier in the file so it runs before any sys.argv access. This prevents the script from crashing and allows it to show the usage message properly.

The original check was around line ~207. It is now placed earlier (around line ~140). Since the early exit block is moved, the elif at the original location is changed to if.

No new logic was added. The same validation code is simply moved earlier to avoid the crash. There are no style or whitespace changes.

Diff summary:

• 1 file changed
• 6 insertions, 5 deletions

Behavior before:
python mkconcore.py → IndexError: list index out of range

Behavior after:
python mkconcore.py → prints usage message and exits cleanly

[Copilot]

Pull request overview

Fixes mkconcore.py crashing with IndexError when invoked without enough CLI arguments by moving the len(sys.argv) validation ahead of any sys.argv[...] access.

Changes:

• Move the existing len(sys.argv) < 4 early-exit usage check to before sys.argv[1], sys.argv[2], and sys.argv[3] are accessed.
• Adjust the later branching from elif to if since the earlier check now handles the missing-args case.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.