@maccesar/titools 2.7.2 → 2.8.0

package/skills/ti-branding/references/master-input-guidelines.md

@@ -1,84 +0,0 @@
-# Master input guidelines
-
-## Preferred: SVG
-
-SVG is the best source because:
-
-- Vector: re-renders cleanly at any density, no upscaling artifacts
-- Preserves transparency natively
-- Small file size (~5–50 KB for typical logos vs 100+ KB for 1024² PNG)
-- Easy to edit by designers in Illustrator / Affinity / Figma / Inkscape
-
-**SVG requirements**:
-
-- Square viewBox (or content that's square within a rectangular viewBox)
-- Paths only — avoid `<script>`, `<foreignObject>`, web fonts
-- Solid fills or simple gradients — very complex gradients may rasterize inconsistently across densities
-- No external references (embedded images, fonts loaded by URL)
-
-The skill uses `rsvg-convert` to rasterize to 1024×1024 PNG before any further processing.
-
-## Acceptable: PNG ≥ 1024×1024 with transparency
-
-A PNG master works when SVG isn't available:
-
-- Minimum size: 1024×1024 (smaller input produces visibly soft xxxhdpi icons)
-- Square preferred — non-square inputs get center-cropped with a warning
-- Transparent background for best monochrome layer results
-
-## Accepted with warning: PNG with solid background
-
-PNGs that already have a non-transparent background still work, but:
-
-- The monochrome layer will be a solid square silhouette (every pixel is non-transparent, so "everything" becomes white)
-- You cannot change the adaptive background color — it's baked in
-- Recommend re-exporting from the source with transparency if possible
-
-## Padding and the "pegado" problem
-
-If the icon's content already fills the full canvas (logo touching the edges), every generated asset will show the same cramped look. The skill cannot add space that isn't there.
-
-Two ways to fix:
-
-1. **Prepare the master with transparent padding** — export the logo so there's ~15% empty space around it in the source
-2. **Increase the `--padding` flag** — the skill re-centers the logo inside the target canvas. Higher padding pushes the logo inward
-
-Recommended padding values:
-
-| Padding | Logo fill | When to use |
-|---------|-----------|-------------|
-| 19 (spec floor) | 62% | Intricate logos needing maximum detail; risky with aggressive launcher masks |
-| 22 (default) | 56% | Balanced, compliant with Android safe-zone spec + 3% buffer |
-| 25 | 50% | Logos with fine strokes or details near the edge |
-| 28–32 | 44–36% | Bold simple shapes, single-letter monograms |
-
-If the user sees the logo pegado after running the skill once, re-run with a higher `--padding`.
-
-## Alpha channel handling
-
-PNG inputs commonly have one of three alpha configurations:
-
-- **Straight alpha** (most common): `rgba(r, g, b, a)` — R/G/B hold true color, A holds coverage
-- **Premultiplied alpha**: `rgba(r × a, g × a, b × a, a)` — R/G/B are scaled by A
-- **No alpha channel**: opaque only
-
-ImageMagick handles straight alpha transparently. Premultiplied inputs may show dark halos at the edges when composited — if this happens, re-export from the source app with straight alpha.
-
-## Checking the master before running
-
-Quick sanity check:
-
-```bash
-# Dimensions and alpha info
-magick identify -format "%wx%h  alpha=%A\n" /path/to/logo.png
-
-# Visualize the alpha channel (useful for debugging halos)
-magick /path/to/logo.png -alpha extract /tmp/alpha.png && open /tmp/alpha.png
-```
-
-For SVG:
-
-```bash
-# Preview at output size
-rsvg-convert -w 1024 -h 1024 /path/to/logo.svg -o /tmp/preview.png && open /tmp/preview.png
-```

package/skills/ti-branding/references/notification-icons.md

@@ -1,63 +0,0 @@
-# Notification icons
-
-## The white-only rule
-
-Android notification icons, since API 21 (Lollipop), must be **white pixels on a transparent background**. The system applies a runtime tint based on the notification's `color` property — color in the source image is discarded.
-
-If the icon is not white+alpha, it renders as a blank white square on API 21+ (a common bug symptom when developers reuse their launcher icon as the notification icon).
-
-## Generation
-
-The skill's `gen-notification.sh`:
-
-1. Scales the foreground logo down to the notification size
-2. Runs ImageMagick's `-channel RGB -fill white -colorize 100 +channel` to force all RGB channels to 255 while preserving alpha
-3. Pads transparently to match the target canvas size
-
-## Sizes
-
-| Density  | Size |
-|----------|------|
-| mdpi     | 24   |
-| hdpi     | 36   |
-| xhdpi    | 48   |
-| xxhdpi   | 72   |
-| xxxhdpi  | 96   |
-
-## File naming
-
-Titanium has no hard convention, but the de-facto name is `ic_stat_notify.png`. The skill emits this name. To reference it:
-
-**In JavaScript (local notification)**:
-
-```js
-Ti.Android.NotificationManager.notify(1, Ti.Android.createNotification({
-  contentTitle: 'Hello',
-  contentText: 'Hi there',
-  icon: Ti.App.Android.R.drawable.ic_stat_notify  // white+alpha
-}))
-```
-
-**In `platform/android/AndroidManifest.xml` (Firebase default)**:
-
-```xml
-<meta-data
-  android:name="com.google.firebase.messaging.default_notification_icon"
-  android:resource="@drawable/ic_stat_notify"/>
-<meta-data
-  android:name="com.google.firebase.messaging.default_notification_color"
-  android:resource="@color/notification_tint"/>
-```
-
-With a matching `values/colors.xml`:
-
-```xml
-<color name="notification_tint">#FDD900</color>
-```
-
-The `default_notification_color` is the tint the runtime applies to the white pixels.
-
-## References
-
-- Android: [Notifications overview](https://developer.android.com/develop/ui/views/notifications)
-- Firebase: [Setting the default notification icon](https://firebase.google.com/docs/cloud-messaging/android/client#manifest)

package/skills/ti-branding/references/splash-screen-api.md

@@ -1,81 +0,0 @@
-# Splash screens (modern)
-
-## Two strategies by platform
-
-### iOS — storyboard-driven (iOS 14+)
-
-Modern iOS uses `LaunchScreen.storyboard`. Titanium enables this via `tiapp.xml`:
-
-```xml
-<ios>
-  <enable-launch-screen-storyboard>true</enable-launch-screen-storyboard>
-  <default-background-color>#0B1326</default-background-color>
-</ios>
-```
-
-The storyboard is a single layout that scales dynamically to any device. No per-device PNG launch images are needed. The `<default-background-color>` sets both the storyboard background AND the gap-color between the storyboard and the first view.
-
-This skill does NOT generate iOS launch images. It prints the `<default-background-color>` snippet for the user to paste.
-
-### Android 12+ — SplashScreen API (API 31+)
-
-Android 12 introduced a system-managed splash that replaces the old "put a PNG in drawable/background.png" approach. It's theme-driven:
-
-```xml
-<!-- platform/android/res/values-v31/splash_theme.xml -->
-<resources>
-  <style name="Theme.SnapGym.SplashScreen" parent="Theme.SplashScreen">
-    <item name="windowSplashScreenBackground">#0B1326</item>
-    <item name="windowSplashScreenAnimatedIcon">@drawable/splash_icon</item>
-    <item name="postSplashScreenTheme">@style/Theme.AppDerived</item>
-  </style>
-</resources>
-```
-
-With a matching `AndroidManifest.xml` entry (inside `<application>`):
-
-```xml
-<meta-data android:name="io.tidev.titanium.splash.theme"
-           android:value="@style/Theme.SnapGym.SplashScreen"/>
-```
-
-Critical: **do NOT apply this theme to `<application android:theme>` or to a specific `<activity>`** — it only affects the system SplashScreen via the manifest meta-data. Applying it to the application breaks the ActionBar / TitleBar in every subsequent screen (as SNAP Gym learned the hard way).
-
-### Android <12 — fallback to launcher icon
-
-On API <31, the system uses the launcher icon as the splash. No extra file needed. The `background.9.png` 9-patch pattern is obsolete in Titanium SDK 13+.
-
-## splash_icon.png canvas (Android 12+)
-
-The SplashScreen API spec:
-
-- Total canvas: **288 dp**
-- Icon safe-zone: **192 dp centered** (≈ 67% of canvas)
-- OS applies a circular mask automatically — keep content centered
-
-Density sizes:
-
-| Density  | Total canvas | Icon area |
-|----------|--------------|-----------|
-| mdpi     | 288          | 192       |
-| hdpi     | 432          | 288       |
-| xhdpi    | 576          | 384       |
-| xxhdpi   | 864          | 576       |
-| xxxhdpi  | 1152         | 768       |
-
-The skill's `gen-splash-icon.sh` places the logo at `192/288 ≈ 67%` of the canvas with transparent padding around it.
-
-## Why we don't generate `background.9.png`
-
-The old 9-patch splash pattern works like this:
-
-- PNG with 1px borders marked as stretchable regions
-- Titanium references it as `background.9.png` in `drawable-*dpi/`
-- SDK pre-13 used it as `windowBackground` for the root activity
-
-Titanium 13+ handles splash via the modern SplashScreen API on 12+ and via the launcher icon itself on older versions. `background.9.png` produces worse results on 12+ (competing with the SplashScreen API) and offers no benefit on older versions that the launcher fallback can't provide.
-
-## References
-
-- Android: [Splash screens](https://developer.android.com/develop/ui/views/launch/splash-screen)
-- Apple: [Configuring your app's launch screen](https://developer.apple.com/documentation/xcode/configuring-your-apps-launch-screen)

package/skills/ti-branding/references/ti-icon-paths.md

@@ -1,84 +0,0 @@
-# Canonical paths — Titanium SDK 13.x
-
-## Alloy layout (primary)
-
-```
-project-root/
-├── DefaultIcon-ios.png              # iOS master (1024² no alpha)
-├── DefaultIcon.png                  # Generic master (1024² no alpha) — fallback
-├── iTunesConnect.png                # 1024² App Store submission
-├── MarketplaceArtwork.png           # 512² Google Play submission
-├── tiapp.xml
-└── app/
-    ├── assets/
-    │   ├── iphone/                  # (optional) manual iOS appiconset — skill doesn't write here
-    │   └── android/
-    │       ├── appicon.png          # 128² legacy fallback (pre-adaptive)
-    │       └── images/              # density-scaled in-app image assets (NOT launcher)
-    └── platform/
-        └── android/
-            └── res/
-                ├── mipmap-mdpi/
-                │   ├── ic_launcher.png              # legacy 48²
-                │   ├── ic_launcher_foreground.png   # adaptive 108²
-                │   ├── ic_launcher_background.png   # adaptive 108²
-                │   └── ic_launcher_monochrome.png   # themed 108²
-                ├── mipmap-hdpi/    # 72 / 162 / 162 / 162
-                ├── mipmap-xhdpi/   # 96 / 216 / 216 / 216
-                ├── mipmap-xxhdpi/  # 144 / 324 / 324 / 324
-                ├── mipmap-xxxhdpi/ # 192 / 432 / 432 / 432
-                ├── mipmap-anydpi-v26/
-                │   └── ic_launcher.xml              # binds the 3 adaptive layers
-                ├── drawable-mdpi/
-                │   ├── ic_stat_notify.png           # (optional) 24² white+alpha
-                │   └── splash_icon.png              # (optional) 288² Android 12+ splash
-                ├── drawable-hdpi/  # 36 / 432
-                ├── drawable-xhdpi/ # 48 / 576
-                ├── drawable-xxhdpi/ # 72 / 864
-                └── drawable-xxxhdpi/ # 96 / 1152
-```
-
-## Classic layout
-
-Same structure, different roots. The skill detects `Resources/` and remaps:
-
-| Alloy path | Classic path |
-|---|---|
-| `app/assets/android/appicon.png` | `Resources/android/appicon.png` |
-| `app/platform/android/res/...` | `platform/android/res/...` |
-
-Root-level files (`DefaultIcon*.png`, `iTunesConnect.png`, `MarketplaceArtwork.png`) are identical in both layouts.
-
-## What Titanium does at build time
-
-- **iOS**: reads the root master → writes to `build/iphone/Assets.xcassets/AppIcon.appiconset/` (build directory, regenerated every build, never commited)
-- **Android**: copies `app/platform/android/res/*` verbatim to `build/android/app/src/main/res/`, plus copies `app/assets/android/appicon.png` to `build/.../res/drawable-*dpi/appicon.png` if the legacy path is used
-
-Result: anything the skill writes to `app/platform/android/res/` or `app/assets/` survives builds. Nothing the skill generates can be accidentally overwritten by `ti build`.
-
-## What the skill writes
-
-After confirming with the user, the skill copies from its staging directory to:
-
-| Target | Files |
-|---|---|
-| Project root | `DefaultIcon-ios.png`, `iTunesConnect.png`, `MarketplaceArtwork.png` |
-| `app/platform/android/res/mipmap-*/` | Launcher icons (adaptive triplet + legacy flat), 5 densities |
-| `app/platform/android/res/mipmap-anydpi-v26/` | `ic_launcher.xml` |
-| `app/platform/android/res/drawable-*/` (optional) | `ic_stat_notify.png`, `splash_icon.png` |
-
-The skill never touches `tiapp.xml`, `config.json`, source code, or anything outside the asset/resource paths above.
-
-## Required `tiapp.xml` entry
-
-For the adaptive icon to be picked up, `tiapp.xml` must reference it:
-
-```xml
-<android>
-  <manifest>
-    <application android:icon="@mipmap/ic_launcher"/>
-  </manifest>
-</android>
-```
-
-The skill checks for this and, if missing, prints the snippet — does not edit the file. See `tiapp-xml-snippets.md` for the full set.

package/skills/ti-branding/references/tiapp-xml-snippets.md

@@ -1,92 +0,0 @@
-# tiapp.xml snippets
-
-The skill prints these blocks after generating assets. The user reviews and pastes them manually — `tiapp.xml` often contains sensitive info (API keys, signing identities, staging URLs) and automated editing would be fragile.
-
-## Block 1 — iOS background color (for storyboard splash)
-
-Under the `<ios>` element:
-
-```xml
-<ios>
-  <enable-launch-screen-storyboard>true</enable-launch-screen-storyboard>
-  <default-background-color>#0B1326</default-background-color>
-</ios>
-```
-
-Replace `#0B1326` with the user's `--bg-color`.
-
-## Block 2 — Android application icon reference
-
-Under `<android><manifest><application>`:
-
-```xml
-<android>
-  <manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    <application android:icon="@mipmap/ic_launcher"
-                 android:usesCleartextTraffic="false"/>
-  </manifest>
-</android>
-```
-
-The `android:icon="@mipmap/ic_launcher"` tells Android to use the adaptive icon XML (via `mipmap-anydpi-v26/ic_launcher.xml` on API 26+) with automatic fallback to the flat `mipmap-*dpi/ic_launcher.png` on older devices.
-
-## Block 3 — Android 12+ splash screen (optional, requires `--with-splash-icon`)
-
-Requires three changes:
-
-### 3a. Splash theme resource
-
-Create `app/platform/android/res/values-v31/splash_theme.xml`:
-
-```xml
-<?xml version="1.0" encoding="utf-8"?>
-<resources>
-  <style name="Theme.App.SplashScreen" parent="Theme.SplashScreen">
-    <item name="windowSplashScreenBackground">#0B1326</item>
-    <item name="windowSplashScreenAnimatedIcon">@drawable/splash_icon</item>
-    <item name="postSplashScreenTheme">@style/Theme.AppDerived</item>
-  </style>
-</resources>
-```
-
-### 3b. Manifest meta-data (NOT a theme on `<application>` or `<activity>`)
-
-Under `<android><manifest><application>`:
-
-```xml
-<meta-data android:name="io.tidev.titanium.splash.theme"
-           android:value="@style/Theme.App.SplashScreen"/>
-```
-
-### 3c. DO NOT do this
-
-Setting `android:theme="@style/Theme.App.SplashScreen"` on `<application>` or `<activity>` will strip the ActionBar / TitleBar from every screen in the app because Titanium's `Theme.AppDerived` inherits from the application theme. The SplashScreen API is designed to work through the manifest meta-data entry above, not through activity theming.
-
-## Block 4 — Firebase Cloud Messaging notification icon (optional, requires `--with-notification`)
-
-Create `app/platform/android/res/values/colors.xml` (if it doesn't exist) with:
-
-```xml
-<?xml version="1.0" encoding="utf-8"?>
-<resources>
-  <color name="notification_tint">#FDD900</color>
-</resources>
-```
-
-Then under `<android><manifest><application>`:
-
-```xml
-<meta-data android:name="com.google.firebase.messaging.default_notification_icon"
-           android:resource="@drawable/ic_stat_notify"/>
-<meta-data android:name="com.google.firebase.messaging.default_notification_color"
-           android:resource="@color/notification_tint"/>
-```
-
-## What the skill reports
-
-After generation, the skill prints only the blocks relevant to the flags used and the project's current state:
-
-- Always prints Block 2 if `android:icon="@mipmap/ic_launcher"` is missing from tiapp.xml
-- Always prints Block 1 if `<default-background-color>` is missing
-- Prints Block 3 only if `--with-splash-icon` was used
-- Prints Block 4 only if `--with-notification` was used

package/skills/ti-branding/scripts/lib/cleanup-legacy.sh

@@ -1,230 +0,0 @@
-#!/usr/bin/env bash
-# cleanup-legacy.sh — context-aware removal of legacy branding artifacts.
-#
-# Reads tiapp.xml to decide what's safe to delete. Categorizes targets into:
-#
-#   SAFE          always deleted — universally obsolete (dead qualifiers etc.)
-#   CONDITIONAL   deleted only when project config guarantees they're unused
-#                 (e.g. iOS legacy launch images when storyboard is enabled)
-#   AGGRESSIVE    behind --aggressive flag — strongly defensible but some
-#                 edge cases remain (e.g. ldpi drawables on <1% of devices)
-#
-# Always prints the plan before acting. Respects --dry-run.
-
-# Append "path|reason" pair to a named array in a bash-3.2-compatible way.
-_add_target() {
-  local array_name="$1"
-  local path="$2"
-  local reason="$3"
-  eval "$array_name+=(\"\$path|\$reason\")"
-}
-
-cleanup_legacy() {
-  local project_root="$1"
-  local project_type="$2"
-  local aggressive="$3"
-  local dry_run="$4"
-
-  local tiapp="$project_root/tiapp.xml"
-  local storyboard_enabled=0
-  local portrait_only=0
-
-  if [[ -f "$tiapp" ]]; then
-    grep -q "<enable-launch-screen-storyboard>true" "$tiapp" && storyboard_enabled=1
-    # Portrait-only heuristic: <orientations> block contains UIInterfaceOrientationPortrait
-    # for iphone AND no Landscape entries.
-    if grep -q '<orientations' "$tiapp" && \
-       ! grep -iqE 'UIInterfaceOrientationLandscape' "$tiapp"; then
-      portrait_only=1
-    fi
-  fi
-
-  local android_images=""
-  local iphone_dir=""
-  local android_assets=""
-  case "$project_type" in
-    alloy)
-      android_images="$project_root/app/assets/android/images"
-      iphone_dir="$project_root/app/assets/iphone"
-      android_assets="$project_root/app/assets/android"
-      ;;
-    classic)
-      android_images="$project_root/Resources/android/images"
-      iphone_dir="$project_root/Resources/iphone"
-      android_assets="$project_root/Resources/android"
-      ;;
-  esac
-
-  local has_adaptive=0
-  [[ -d "$project_root/app/platform/android/res/mipmap-anydpi-v26" ]] && has_adaptive=1
-  [[ -d "$project_root/platform/android/res/mipmap-anydpi-v26" ]] && has_adaptive=1
-
-  local safe_targets=()
-  local cond_targets=()
-  local aggressive_targets=()
-
-  # -------------------------------------------------------------------------
-  # SAFE
-  # -------------------------------------------------------------------------
-
-  if [[ -n "$android_images" && -d "$android_images" ]]; then
-    local d
-    for d in "$android_images"/res-long-* "$android_images"/res-notlong-*; do
-      [[ -d "$d" ]] && _add_target safe_targets "$d" "Android long/notlong qualifier (dead since Android 3.0, 2011)"
-    done
-  fi
-
-  # -------------------------------------------------------------------------
-  # CONDITIONAL
-  # -------------------------------------------------------------------------
-
-  # iOS legacy launch images — only safe with storyboard enabled
-  if [[ $storyboard_enabled -eq 1 && -n "$iphone_dir" && -d "$iphone_dir" ]]; then
-    local f
-    for f in "$iphone_dir"/Default-*.png "$iphone_dir"/Default@2x.png; do
-      [[ -f "$f" ]] && _add_target cond_targets "$f" "iOS legacy launch image (storyboard enabled → not consulted)"
-    done
-  fi
-
-  # Android landscape-variant folders — safe when app is portrait-only
-  if [[ $portrait_only -eq 1 && -n "$android_images" && -d "$android_images" ]]; then
-    local d
-    for d in "$android_images"/res-*-land-* "$android_images"/res-land-*; do
-      [[ -d "$d" ]] && _add_target cond_targets "$d" "Landscape variant (app is portrait-only)"
-    done
-  fi
-
-  # Android legacy splash PNG — safe when adaptive icons are in place
-  if [[ $has_adaptive -eq 1 && -n "$android_assets" && -f "$android_assets/default.png" ]]; then
-    _add_target cond_targets "$android_assets/default.png" "Legacy Android splash PNG (adaptive launcher handles splash now)"
-  fi
-
-  # Android legacy appicon.png — redundant when adaptive icons are in place
-  if [[ $has_adaptive -eq 1 && -n "$android_assets" && -f "$android_assets/appicon.png" ]]; then
-    _add_target cond_targets "$android_assets/appicon.png" "Legacy appicon.png (adaptive launcher takes precedence)"
-  fi
-
-  # -------------------------------------------------------------------------
-  # AGGRESSIVE
-  # -------------------------------------------------------------------------
-
-  if [[ $aggressive -eq 1 ]]; then
-    # ldpi everywhere — <1% of active devices globally in 2026
-    local d
-    if [[ -n "$android_images" && -d "$android_images" ]]; then
-      for d in "$android_images"/res-ldpi "$android_images"/res-*-ldpi; do
-        [[ -d "$d" ]] && _add_target aggressive_targets "$d" "ldpi density (<1% global market)"
-      done
-    fi
-    for d in "$project_root/app/platform/android/res/drawable-ldpi" \
-             "$project_root/app/platform/android/res/values-ldpi" \
-             "$project_root/app/platform/android/res/mipmap-ldpi" \
-             "$project_root/platform/android/res/drawable-ldpi" \
-             "$project_root/platform/android/res/values-ldpi" \
-             "$project_root/platform/android/res/mipmap-ldpi"; do
-      [[ -d "$d" ]] && _add_target aggressive_targets "$d" "ldpi resource folder (<1% global market)"
-    done
-  fi
-
-  # -------------------------------------------------------------------------
-  # Print plan
-  # -------------------------------------------------------------------------
-
-  echo
-  echo "${C_YELLOW}⚠  WARNING${C_RESET} — --cleanup-legacy deletes files permanently."
-  echo "   Recommended: commit your project with git before running without --dry-run."
-  echo "   Provided AS IS under the MIT License. No warranty — use at your own risk."
-  echo
-  echo "${C_BOLD}Cleanup plan${C_RESET}"
-  echo "  ${C_DIM}Project:            $project_root ($project_type)${C_RESET}"
-  echo "  ${C_DIM}Storyboard:         $([[ $storyboard_enabled -eq 1 ]] && echo "enabled" || echo "disabled")${C_RESET}"
-  echo "  ${C_DIM}Orientation:        $([[ $portrait_only -eq 1 ]] && echo "portrait-only" || echo "landscape allowed")${C_RESET}"
-  echo "  ${C_DIM}Adaptive icons:     $([[ $has_adaptive -eq 1 ]] && echo "present" || echo "not detected")${C_RESET}"
-  echo "  ${C_DIM}Aggressive mode:    $([[ $aggressive -eq 1 ]] && echo "on (includes ldpi)" || echo "off")${C_RESET}"
-
-  local total=$(( ${#safe_targets[@]} + ${#cond_targets[@]} + ${#aggressive_targets[@]} ))
-
-  if [[ $total -eq 0 ]]; then
-    echo
-    log_ok "No legacy artifacts detected. Project is already clean."
-    return 0
-  fi
-
-  # bash 3.2 under `set -u` errors on "${empty_array[@]}" — iterate per-bucket
-  # with explicit length guards instead of expanding empty arrays into args.
-  _print_bucket_safe() {
-    local t path reason size
-    echo
-    echo "${C_GREEN}SAFE — universally obsolete${C_RESET}"
-    for t in "${safe_targets[@]}"; do
-      path="${t%%|*}"; reason="${t#*|}"
-      size=$(du -sk "$path" 2>/dev/null | awk '{print $1}')
-      [[ -z "$size" ]] && size=0
-      printf "    %-6sK  %s\n" "$size" "${path#$project_root/}"
-      printf "              ${C_DIM}%s${C_RESET}\n" "$reason"
-    done
-  }
-  _print_bucket_cond() {
-    local t path reason size
-    echo
-    echo "${C_YELLOW}CONDITIONAL — safe given your project config${C_RESET}"
-    for t in "${cond_targets[@]}"; do
-      path="${t%%|*}"; reason="${t#*|}"
-      size=$(du -sk "$path" 2>/dev/null | awk '{print $1}')
-      [[ -z "$size" ]] && size=0
-      printf "    %-6sK  %s\n" "$size" "${path#$project_root/}"
-      printf "              ${C_DIM}%s${C_RESET}\n" "$reason"
-    done
-  }
-  _print_bucket_aggr() {
-    local t path reason size
-    echo
-    echo "${C_RED}AGGRESSIVE — --aggressive enabled${C_RESET}"
-    for t in "${aggressive_targets[@]}"; do
-      path="${t%%|*}"; reason="${t#*|}"
-      size=$(du -sk "$path" 2>/dev/null | awk '{print $1}')
-      [[ -z "$size" ]] && size=0
-      printf "    %-6sK  %s\n" "$size" "${path#$project_root/}"
-      printf "              ${C_DIM}%s${C_RESET}\n" "$reason"
-    done
-  }
-
-  [[ ${#safe_targets[@]} -gt 0 ]] && _print_bucket_safe
-  [[ ${#cond_targets[@]} -gt 0 ]] && _print_bucket_cond
-  [[ ${#aggressive_targets[@]} -gt 0 ]] && _print_bucket_aggr
-
-  echo
-  echo "  ${C_BOLD}Total: $total item(s) to remove${C_RESET}"
-
-  if [[ $dry_run -eq 1 ]]; then
-    echo
-    log_info "Dry-run mode — nothing deleted. Re-run without --dry-run to apply."
-    return 0
-  fi
-
-  # -------------------------------------------------------------------------
-  # Apply
-  # -------------------------------------------------------------------------
-
-  echo
-  log_step "Applying cleanup"
-  local t path
-  if [[ ${#safe_targets[@]} -gt 0 ]]; then
-    for t in "${safe_targets[@]}"; do
-      path="${t%%|*}"; rm -rf "$path"
-      log_ok "Removed ${path#$project_root/}"
-    done
-  fi
-  if [[ ${#cond_targets[@]} -gt 0 ]]; then
-    for t in "${cond_targets[@]}"; do
-      path="${t%%|*}"; rm -rf "$path"
-      log_ok "Removed ${path#$project_root/}"
-    done
-  fi
-  if [[ ${#aggressive_targets[@]} -gt 0 ]]; then
-    for t in "${aggressive_targets[@]}"; do
-      path="${t%%|*}"; rm -rf "$path"
-      log_ok "Removed ${path#$project_root/}"
-    done
-  fi
-}