[BUG] check_zone_fills preflight silently prevents checking schematic parity

[adamjsmithnz]

Describe the bug
Running the check_zone_fills preflight silently prevents checking for schematic parity warnings. Wondering if this is known or expected behavior, because this was unexpected for me, and I couldn't find any other references to it.

To Reproduce
See https://github.com/adamjsmithnz/kibot-parity-test for a minimal example.
With check_zone_fills: true removed, parity warnings are correctly shown. With it uncommented, it reports 0 warnings and 0 errors.

Expected behavior
Parity checks run when check_zone_fills is used, or a warning is displayed saying they are incompatible.

Environment
Using the latest KiCad 9 Docker image: ghcr.io/inti-cmnb/kicad9_auto_full:latest

Additional context
Running kibot --verbose shows the temporary project used to fill zones does not have part annotations set correctly, preventing kicad-cli from checking for parity. Relevant logs:

DEBUG:Writing modified PCB to /app/kibot_naagsysn.kicad_pcb (kibot.pre_drc - gs.py:1036)
DEBUG:Copying project `/app/kibot-parity-test.kicad_pro` to `/app/kibot_naagsysn.kicad_pro` (kibot.gs - gs.py:582)
DEBUG:Copying project local settings `/app/kibot-parity-test.kicad_prl` to `/app/kibot_naagsysn.kicad_prl` (kibot.gs - gs.py:590)
- Running the DRC
DEBUG:- Executing: kicad-cli pcb drc -o /app/kibot/kibot-parity-test-drc.html --format json --severity-all --units mm --schematic-parity /app/kibot_naagsysn.kicad_pcb (kibot - kiplot.py:159)
DEBUG:- Output from command: Failed to fetch schematic netlist for parity tests.
Schematic parity tests require a fully annotated schematic.

[adamjsmithnz]

With a closer look, the copy_project method copies the kicad_pro, kicad_prl, and kicad_dru files but not the kicad_sch:

KiBot/kibot/gs.py

Lines 581 to 605 in 367a2e0

	def copy_project(new_pcb_name, dry=False):
	pro_name = GS.pro_file
	if pro_name is None or not os.path.isfile(pro_name):
	return None, None, None
	pro_copy = os.path.splitext(new_pcb_name)[0]+GS.pro_ext
	if not dry:
	logger.debug(f'Copying project `{pro_name}` to `{pro_copy}`')
	copy2(pro_name, pro_copy)
	# Also copy the PRL
	prl_name = pro_name[:-3]+'prl'
	prl_copy = None
	if os.path.isfile(prl_name):
	prl_copy = pro_copy[:-3]+'prl'
	if not dry:
	logger.debug(f'Copying project local settings `{prl_name}` to `{prl_copy}`')
	copy2(prl_name, prl_copy)
	# ... and the DRU
	dru_name = pro_name[:-3]+'dru'
	dru_copy = None
	if os.path.isfile(dru_name):
	dru_copy = pro_copy[:-3]+'dru'
	if not dry:
	logger.debug(f'Copying project custom design rules `{dru_name}` to `{dru_copy}`')
	copy2(dru_name, dru_copy)
	return pro_copy, prl_copy, dru_copy

Adding a section to copy the schematic as well solves the problem for me. Does this seem like a reasonable fix?

[set-soft]

Hi @adamjsmithnz !
Copying the schematic isn't trivial, sub-sheets might be anywhere.
I'll try another approach.

[set-soft]

Hi @adamjsmithnz !
The above patch should solve the issue.
Now KiBot is moving the current PCB to a back-up, saves the file with refilled areas, runs the DRC and then restores the original file. The file is restored even if something goes wrong during DRC.