Fixes #980 - Fix empty items_to_include list causing full deployment

[Copilot]

Description

Passing items_to_include=[] to publish_all_items deploys everything instead of nothing. Root cause: truthiness checks (if not items_to_include) treat None and [] identically, but they have different semantics—None means "no filter" while [] means "include nothing."

Fixes:

• publish.py: Three checks changed from truthiness (if items_to_include:, if not items_to_include) to explicit is None checks
• _base_publisher.py: get_items_to_publish() now returns empty dict for [] instead of all items
• test_publish.py: Added test_empty_items_to_include_skips_all_items

# Before: [] is falsy, so this block is skipped — items_to_include never set on workspace
if items_to_include:
    fabric_workspace_obj.items_to_include = items_to_include

# After: only skip when caller didn't pass the parameter at all
if items_to_include is not None:
    fabric_workspace_obj.items_to_include = items_to_include

Linked Issue (REQUIRED)

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• releases.astral.sh
  • Triggering command: /home/REDACTED/.local/bin/uv uv sync --dev (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[Copilot]

Pull request overview

This PR fixes a behavioral bug in selective publishing/unpublishing where passing items_to_include=[] was treated the same as None, causing the library to deploy (or consider) all items instead of none. It aligns items_to_include handling with the established semantics: None = no filter, [] = include nothing.

Changes:

• Update items_to_include checks from truthiness to explicit is None checks in publish_all_items() and unpublish_all_orphan_items().
• Fix core publisher filtering so items_to_include=[] returns an empty publish set (instead of all items).
• Add a regression test ensuring items_to_include=[] results in all repository items being marked skip_publish=True.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/fabric_cicd/publish.py	Uses explicit is None checks so [] is treated as “include nothing” rather than “no filter”.
src/fabric_cicd/_items/_base_publisher.py	Fixes get_items_to_publish() to return an empty dict for items_to_include=[].
tests/test_publish.py	Adds regression coverage for items_to_include=[] skipping all items.
src/fabric_cicd/fabric_workspace.py	Minor formatting change (single-line endpoint call).
.changes/unreleased/fixed-20260513-105123.yaml	Adds changelog fragment describing the fix.