Add 30 manual verification flows for comprehensive CLI testing #2

akiselev · 2026-01-28T20:32:37Z

Covers all major functionality areas: SchLib (6 flows), PcbLib (6 flows),
SchDoc editing and analysis (8 flows), PcbDoc board operations (5 flows),
PrjPcb project management (2 flows), query languages (2 flows), and
cross-cutting features like inspect, JSON output, and shell completions (1 flow).

https://claude.ai/code/session_01WwDHQFPm8vrsJoS2ptPmSf

Covers all major functionality areas: SchLib (6 flows), PcbLib (6 flows), SchDoc editing and analysis (8 flows), PcbDoc board operations (5 flows), PrjPcb project management (2 flows), query languages (2 flows), and cross-cutting features like inspect, JSON output, and shell completions (1 flow). https://claude.ai/code/session_01WwDHQFPm8vrsJoS2ptPmSf

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 11a6c8896a

ℹ️ 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".

chatgpt-codex-connector · 2026-01-28T20:38:09Z

VERIFICATION_FLOWS.md

+cargo run -p altium-cli -- schlib add-line /tmp/test_flow3.SchLib "IC1" -30 0 -40 0
+
+# 4. Add a polygon
+cargo run -p altium-cli -- schlib add-polygon /tmp/test_flow3.SchLib "IC1" "0,50 -10,40 10,40"


Use comma-only vertex list for schlib add-polygon

In cmd_add_polygon (crates/altium-format/src/ops/schlib.rs) the vertices string is parsed by splitting on commas only, so the example "0,50 -10,40 10,40" yields tokens like "50 -10" that fail unit parsing and the command errors. As written, Flow 3 will stop at step 4; use a flat comma-separated list like 0,50,-10,40,10,40 or insert commas between every number.

Useful? React with 👍 / 👎.

chatgpt-codex-connector · 2026-01-28T20:38:09Z

VERIFICATION_FLOWS.md

+cargo run -p altium-cli -- schlib create /tmp/test_flow4.SchLib
+
+# 2. Generate IC symbol with named pins
+cargo run -p altium-cli -- schlib gen-ic /tmp/test_flow4.SchLib "ATtiny85" --pins "VCC,PB0,PB1,PB2,PB3,PB4,RESET,GND"


Fix gen-ic pin definition format

The schlib gen-ic parser (parse_pin_defs in crates/altium-format/src/ops/schlib.rs and the GenIc CLI docs in crates/altium-cli/src/commands/schlib.rs) requires each pin as designator:name:type[:side]. Passing only names (VCC,PB0,…) triggers “Invalid pin spec” and the flow cannot proceed; use e.g. 1:VCC:power:left etc. This same pattern appears in later flows that use gen-ic.

Useful? React with 👍 / 👎.

chatgpt-codex-connector · 2026-01-28T20:38:09Z

VERIFICATION_FLOWS.md

+# 3. Add from JSON
+cargo run -p altium-cli -- schlib add-json /tmp/test_flow6.SchLib /tmp/component_def.json


Pass JSON file via --file for add-json

SchLibCommands::AddJson accepts only --file or --json (crates/altium-cli/src/commands/schlib.rs); it does not take a positional JSON path. This invocation will be rejected as an unexpected argument, so Flow 6 can’t import the component. It should be schlib add-json … --file /tmp/component_def.json (same issue appears in Flow 12 for pcblib add-json).

Useful? React with 👍 / 👎.

chatgpt-codex-connector · 2026-01-28T20:38:09Z

VERIFICATION_FLOWS.md

+
+# 2. Add footprint and generate dual-row pads (SOIC-8 style)
+cargo run -p altium-cli -- pcblib add-footprint /tmp/test_flow10.PcbLib "SOIC-8" -d "8-pin SOIC"
+cargo run -p altium-cli -- pcblib add-dual-row /tmp/test_flow10.PcbLib -f "SOIC-8" --pitch 50 --span 240 --start 1 -n 8 -w 25 --height 60


Update pcblib pad-row flags to match CLI

pcblib add-dual-row does not accept --span, --start, -w, or --height; the CLI expects --row-spacing plus --pad-width/--pad-height (or --pad-diameter/--hole) per PcbLibCommands::AddDualRow in crates/altium-cli/src/commands/pcblib.rs. As written, clap will exit with unknown-argument errors and Flow 10 won’t generate pads (the same flag mix-up appears in Flow 11).

Useful? React with 👍 / 👎.

chatgpt-codex-connector · 2026-01-28T20:38:09Z

VERIFICATION_FLOWS.md

+cargo run -p altium-cli -- pcbdoc add-net /tmp/test_flow23.PcbDoc "SIG1"
+
+# 3. Place components
+cargo run -p altium-cli -- pcbdoc place-component /tmp/test_flow23.PcbDoc "U1" 500 750


Use --at for pcbdoc place-component positions

PcbDocCommands::PlaceComponent (crates/altium-cli/src/commands/pcbdoc.rs) requires the position via --at x,y (or alternatives like --near, --edge); positional coordinates aren’t accepted. This line will be parsed as extra args and fail, preventing Flow 23 from placing components. Use --at 500,750 instead.

Useful? React with 👍 / 👎.

chatgpt-codex-connector · 2026-01-28T20:38:09Z

VERIFICATION_FLOWS.md

+# 4. Add track segment
+cargo run -p altium-cli -- pcbdoc add-track /tmp/test_flow23.PcbDoc --start 500,750 --end 1000,750 -w 10 -l top -n "SIG1"
+
+# 5. Add via


Fix pcbdoc add-via flags and required layers

PcbDocCommands::AddVia expects --at (or --at-pad), --diameter, and mandatory --from-layer/--to-layer (crates/altium-cli/src/commands/pcbdoc.rs); it does not recognize --x, --y, or --size. As written, this step will error on unknown args/missing required flags and the via won’t be created in Flow 23.

Useful? React with 👍 / 👎.

Copilot

Pull request overview

Adds a comprehensive set of manual verification flows intended to exercise the majority of altium-cli functionality end-to-end via shell commands.

Changes:

Introduces 30 self-contained manual verification flows spanning SchLib, PcbLib, SchDoc, PcbDoc, PrjPcb, query features, and cross-cutting CLI behavior.
Defines expected “pass criteria” per flow and includes a category summary table for coverage tracking.

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

Copilot · 2026-01-28T20:45:29Z

VERIFICATION_FLOWS.md

+cargo run -p altium-cli -- schlib add-pin /tmp/test_flow2.SchLib "LM7805" "1" "VIN" -50 0 -e input -o right
+cargo run -p altium-cli -- schlib add-pin /tmp/test_flow2.SchLib "LM7805" "2" "GND" 0 -50 -e power -o up
+cargo run -p altium-cli -- schlib add-pin /tmp/test_flow2.SchLib "LM7805" "3" "VOUT" 50 0 -e output -o left


The schlib add-pin examples pass negative X/Y as positional args (e.g., -50). In the current CLI, x/y are positional strings without allow_hyphen_values, so values starting with - are parsed as flags and the command will fail. Rework these examples to avoid leading - positionals (or move coordinates after options and use -- to terminate option parsing).

Suggested change

cargo run -p altium-cli -- schlib add-pin /tmp/test_flow2.SchLib "LM7805" "1" "VIN" -50 0 -e input -o right

cargo run -p altium-cli -- schlib add-pin /tmp/test_flow2.SchLib "LM7805" "2" "GND" 0 -50 -e power -o up

cargo run -p altium-cli -- schlib add-pin /tmp/test_flow2.SchLib "LM7805" "3" "VOUT" 50 0 -e output -o left

cargo run -p altium-cli -- schlib add-pin /tmp/test_flow2.SchLib "LM7805" "1" "VIN" -e input -o right -- -50 0

cargo run -p altium-cli -- schlib add-pin /tmp/test_flow2.SchLib "LM7805" "2" "GND" -e power -o up -- 0 -50

cargo run -p altium-cli -- schlib add-pin /tmp/test_flow2.SchLib "LM7805" "3" "VOUT" -e output -o left -- 50 0

Copilot · 2026-01-28T20:45:29Z

VERIFICATION_FLOWS.md

+cargo run -p altium-cli -- schlib add-rectangle /tmp/test_flow3.SchLib "IC1" -30 -40 30 40 --filled
+
+# 3. Add a line
+cargo run -p altium-cli -- schlib add-line /tmp/test_flow3.SchLib "IC1" -30 0 -40 0


These schlib add-rectangle/add-line examples also use negative positional coordinates (e.g., -30, -40). With the current CLI, those positionals will be interpreted as flags and the commands won’t parse. Please adjust the examples to avoid leading - positionals (or restructure to use -- after options).

Suggested change

cargo run -p altium-cli -- schlib add-rectangle /tmp/test_flow3.SchLib "IC1" -30 -40 30 40 --filled

# 3. Add a line

cargo run -p altium-cli -- schlib add-line /tmp/test_flow3.SchLib "IC1" -30 0 -40 0

cargo run -p altium-cli -- schlib add-rectangle /tmp/test_flow3.SchLib "IC1" --filled -- -30 -40 30 40

# 3. Add a line

cargo run -p altium-cli -- schlib add-line /tmp/test_flow3.SchLib "IC1" -- -30 0 -40 0

Copilot · 2026-01-28T20:45:30Z

VERIFICATION_FLOWS.md

+  ]
+}
+EOF
+cargo run -p altium-cli -- pcblib add-json /tmp/test_flow12b.PcbLib /tmp/footprint_def.json


pcblib add-json does not accept the JSON filepath as a positional argument. It requires --file/-f <path> (or --json/-j <string>), so this command will fail as written.

Suggested change

cargo run -p altium-cli -- pcblib add-json /tmp/test_flow12b.PcbLib /tmp/footprint_def.json

cargo run -p altium-cli -- pcblib add-json /tmp/test_flow12b.PcbLib --file /tmp/footprint_def.json

Copilot · 2026-01-28T20:45:30Z

VERIFICATION_FLOWS.md

+cargo run -p altium-cli -- pcbdoc place-component /tmp/test_flow23.PcbDoc "U1" 500 750
+cargo run -p altium-cli -- pcbdoc place-component /tmp/test_flow23.PcbDoc "R1" 1000 750
+cargo run -p altium-cli -- pcbdoc place-component /tmp/test_flow23.PcbDoc "C1" 1500 750


pcbdoc place-component takes an optional --at "x,y" (and other placement options); it does not take X and Y as positional arguments. These commands will fail as written.

Suggested change

cargo run -p altium-cli -- pcbdoc place-component /tmp/test_flow23.PcbDoc "U1" 500 750

cargo run -p altium-cli -- pcbdoc place-component /tmp/test_flow23.PcbDoc "R1" 1000 750

cargo run -p altium-cli -- pcbdoc place-component /tmp/test_flow23.PcbDoc "C1" 1500 750

cargo run -p altium-cli -- pcbdoc place-component /tmp/test_flow23.PcbDoc "U1" --at "500,750"

cargo run -p altium-cli -- pcbdoc place-component /tmp/test_flow23.PcbDoc "R1" --at "1000,750"

cargo run -p altium-cli -- pcbdoc place-component /tmp/test_flow23.PcbDoc "C1" --at "1500,750"

Copilot · 2026-01-28T20:45:30Z

VERIFICATION_FLOWS.md

+# 2. Add several rules
+cargo run -p altium-cli -- pcbdoc add-rule /tmp/test_flow25.PcbDoc clearance --value 6
+cargo run -p altium-cli -- pcbdoc add-rule /tmp/test_flow25.PcbDoc width --value 8
+cargo run -p altium-cli -- pcbdoc add-rule /tmp/test_flow25.PcbDoc via --value 20
+


Flow 25’s pcbdoc add-rule examples use the wrong argument shape (missing required rule name, and --value is not a supported flag). These commands won’t parse with the current CLI.

Copilot · 2026-01-28T20:45:34Z

VERIFICATION_FLOWS.md

+cargo run -p altium-cli -- pcbdoc add-polygon /tmp/test_flow24.PcbDoc top "GND" "0,0 3000,0 3000,2000 0,2000"
+
+# 5. Add text on silkscreen
+cargo run -p altium-cli -- pcbdoc add-text /tmp/test_flow24.PcbDoc --text "REV A" --x 1500 --y 100 --layer top-overlay --height 50


pcbdoc add-text expects the text and position as positional args (text and at in "x,y" form) plus --layer/-l; it does not support --text/--x/--y. This command will fail as written.

Suggested change

cargo run -p altium-cli -- pcbdoc add-text /tmp/test_flow24.PcbDoc --text "REV A" --x 1500 --y 100 --layer top-overlay --height 50

cargo run -p altium-cli -- pcbdoc add-text /tmp/test_flow24.PcbDoc "REV A" 1500,100 --layer top-overlay --height 50

Copilot · 2026-01-28T20:45:34Z

VERIFICATION_FLOWS.md

+
+# 4. Add BGA-style grid pads
+cargo run -p altium-cli -- pcblib add-footprint /tmp/test_flow10.PcbLib "BGA-16" -d "16-ball BGA"
+cargo run -p altium-cli -- pcblib add-pad-grid /tmp/test_flow10.PcbLib -f "BGA-16" --pitch 50 --rows 4 --cols 4 -w 25 --height 25


The pcblib add-pad-grid example uses flags that don't exist (-w, --height). The current CLI takes --pad-diameter (and optional --shape/--skip-center) for grid pads, so this command will fail as written.

Suggested change

cargo run -p altium-cli -- pcblib add-pad-grid /tmp/test_flow10.PcbLib -f "BGA-16" --pitch 50 --rows 4 --cols 4 -w 25 --height 25

cargo run -p altium-cli -- pcblib add-pad-grid /tmp/test_flow10.PcbLib -f "BGA-16" --pitch 50 --rows 4 --cols 4 --pad-diameter 25

Copilot · 2026-01-28T20:45:34Z

VERIFICATION_FLOWS.md

+cargo run -p altium-cli -- schlib gen-ic /tmp/test_flow14.SchLib "LM7805" --pins "VIN,GND,VOUT"
+cargo run -p altium-cli -- schlib gen-ic /tmp/test_flow14.SchLib "RES" --pins "1,2"


Same issue as Flow 4: schlib gen-ic takes the pins string as a positional argument, and the pins must be in designator:name:type[:side],... format. Using --pins "VIN,GND,VOUT" will not parse and the abbreviated pin list format is invalid.

Suggested change

cargo run -p altium-cli -- schlib gen-ic /tmp/test_flow14.SchLib "LM7805" --pins "VIN,GND,VOUT"

cargo run -p altium-cli -- schlib gen-ic /tmp/test_flow14.SchLib "RES" --pins "1,2"

cargo run -p altium-cli -- schlib gen-ic /tmp/test_flow14.SchLib "LM7805" "1:VIN:pwr:LEFT,2:GND:pwr:LEFT,3:VOUT:pwr:RIGHT"

cargo run -p altium-cli -- schlib gen-ic /tmp/test_flow14.SchLib "RES" "1:1:passive:LEFT,2:2:passive:RIGHT"

Copilot · 2026-01-28T20:45:35Z

VERIFICATION_FLOWS.md

+```bash
+# 1. Setup: Create schematic with library and placed component
+cargo run -p altium-cli -- schlib create /tmp/test_flow16.SchLib
+cargo run -p altium-cli -- schlib gen-ic /tmp/test_flow16.SchLib "U1_IC" --pins "A,B,C,D"


schlib gen-ic here uses --pins and an abbreviated pins list, but the CLI expects a positional pins string in designator:name:type[:side],... format. As written, this command won’t parse.

Suggested change

cargo run -p altium-cli -- schlib gen-ic /tmp/test_flow16.SchLib "U1_IC" --pins "A,B,C,D"

cargo run -p altium-cli -- schlib gen-ic /tmp/test_flow16.SchLib "U1_IC" "1:A:io,2:B:io,3:C:io,4:D:io"

Copilot · 2026-01-28T20:45:35Z

VERIFICATION_FLOWS.md

+```bash
+# 1. Use the blank schematic as base, add content
+cargo run -p altium-cli -- schlib create /tmp/test_flow20.SchLib
+cargo run -p altium-cli -- schlib gen-ic /tmp/test_flow20.SchLib "OPAMP" --pins "V+,V-,OUT,VCC,VEE"


schlib gen-ic does not support --pins, and the pins string must be in designator:name:type[:side],... format. This invocation will fail as written.

Suggested change

cargo run -p altium-cli -- schlib gen-ic /tmp/test_flow20.SchLib "OPAMP" --pins "V+,V-,OUT,VCC,VEE"

cargo run -p altium-cli -- schlib gen-ic /tmp/test_flow20.SchLib "OPAMP" "1:V+:pwr:left,2:V-:pwr:left,3:OUT:io:right,4:VCC:pwr:top,5:VEE:pwr:bottom"

Copilot AI review requested due to automatic review settings January 28, 2026 20:32

Copilot started reviewing on behalf of akiselev January 28, 2026 20:32 View session

chatgpt-codex-connector bot reviewed Jan 28, 2026

View reviewed changes

Copilot AI reviewed Jan 28, 2026

View reviewed changes

		# 3. Add from JSON
		cargo run -p altium-cli -- schlib add-json /tmp/test_flow6.SchLib /tmp/component_def.json

	cargo run -p altium-cli -- pcblib add-json /tmp/test_flow12b.PcbLib /tmp/footprint_def.json
	cargo run -p altium-cli -- pcblib add-json /tmp/test_flow12b.PcbLib --file /tmp/footprint_def.json

	cargo run -p altium-cli -- pcbdoc add-text /tmp/test_flow24.PcbDoc --text "REV A" --x 1500 --y 100 --layer top-overlay --height 50
	cargo run -p altium-cli -- pcbdoc add-text /tmp/test_flow24.PcbDoc "REV A" 1500,100 --layer top-overlay --height 50

	cargo run -p altium-cli -- pcblib add-pad-grid /tmp/test_flow10.PcbLib -f "BGA-16" --pitch 50 --rows 4 --cols 4 -w 25 --height 25
	cargo run -p altium-cli -- pcblib add-pad-grid /tmp/test_flow10.PcbLib -f "BGA-16" --pitch 50 --rows 4 --cols 4 --pad-diameter 25

		cargo run -p altium-cli -- schlib gen-ic /tmp/test_flow14.SchLib "LM7805" --pins "VIN,GND,VOUT"
		cargo run -p altium-cli -- schlib gen-ic /tmp/test_flow14.SchLib "RES" --pins "1,2"

Add 30 manual verification flows for comprehensive CLI testing #2

Are you sure you want to change the base?

Add 30 manual verification flows for comprehensive CLI testing #2

Uh oh!

Conversation

akiselev commented Jan 28, 2026

Uh oh!

chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector bot Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

chatgpt-codex-connector bot Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

chatgpt-codex-connector bot Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

chatgpt-codex-connector bot Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

chatgpt-codex-connector bot Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

chatgpt-codex-connector bot Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants