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

[GaneshPatil7517]

@pradeeban sir

Summary

Fixes #267 — mkconcore.py crashes with IndexError when run without arguments because sys.argv[1], sys.argv[2], and sys.argv[3] are accessed before the argument count is validated.

Changes

• Moved the existing len(sys.argv) < 4 check from line ~207 to line ~140, before any sys.argv access
• Changed elif to if at the original location since the early-exit block was removed above it

No new logic, no style changes, no whitespace modifications. The same validation code is simply relocated earlier.

Diff

• 1 file changed
• 6 insertions, 5 deletions

Before / After

Before: python mkconcore.py → IndexError: list index out of range
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.