@ozzylabs/feedradar 0.1.7 → 0.1.9

package/dist/templates/workflows/combined-with-triage.template.yaml.tmpl

@@ -5,6 +5,14 @@
 # LoC-heavy research/review steps see ~5-15% of the raw detected count
 # instead of the full firehose. Template placeholders / hard-cap rationale
 # are documented in `src/cli/workflow/generate-combined-with-triage.ts`.
+#
+# The final step is rendered from `--output-mode` (default `pr`):
+#   pr            — peter-evans/create-pull-request@v6 (human reviews the PR
+#                   before research/ lands on the default branch).
+#   direct-commit — commit & push straight to the default branch (no PR gate;
+#                   `permissions:` drops `pull-requests: write`). Do NOT put a
+#                   PR-required branch protection on the default branch when
+#                   using this mode — the bot pushes directly.
 
 name: feedradar-daily
 
@@ -13,9 +21,7 @@ on:
     - cron: "{{watchCron}}"
   workflow_dispatch: {}
 
-permissions:
-  contents: write
-  pull-requests: write
+{{permissionsBlock}}
 
 concurrency:
   # Distinct from feedradar-watch / feedradar-combined so this can co-exist
@@ -70,19 +76,33 @@ jobs:
         # matching item.
         run: |
           set -euo pipefail
-          GROUPS=$(radar items list --status triaged_digest --field triage.group | sort -u | grep -v '^-$' || true)
-          if [ -z "$GROUPS" ]; then
+          # `mapfile` reads the group list into an array so each element is a
+          # single, properly-quoted token. The earlier command-substitution
+          # scalar assignment tripped actionlint/shellcheck SC2128 (expanding an
+          # array without an index) / SC2178 (reusing a name as both scalar and
+          # array) once the loop below treated it array-like. The pipeline still
+          # de-dupes and drops the `-` sentinel that `--field` prints for
+          # ungrouped items.
+          #
+          # NB: the array is NOT named `GROUPS` — that is a special bash array
+          # (the caller's group IDs); reading the group list into that reserved
+          # name fails outright under `set -e`. Use a non-reserved name.
+          mapfile -t DIGEST_GROUPS < <(radar items list --status triaged_digest --field triage.group | sort -u | grep -v '^-$' || true)
+          if [ "${#DIGEST_GROUPS[@]}" -eq 0 ]; then
             echo "no triaged_digest groups to process"
             exit 0
           fi
-          while IFS= read -r GROUP; do
+          for GROUP in "${DIGEST_GROUPS[@]}"; do
             [ -z "$GROUP" ] && continue
             echo "::group::digest for triage.group=$GROUP"
             IDS=$(radar items list --triage-group "$GROUP" --status triaged_digest --field id | tr '\n' ' ')
+            # `--triage-group "$GROUP"` names the digest file after the group so
+            # two same-day groups from a single-keyword source no longer collide
+            # on `<date>_digest_<slug>_v1.md` (#255).
             # shellcheck disable=SC2086
-            radar research --digest $IDS --agent {{researchAgent}}
+            radar research --digest $IDS --triage-group "$GROUP" --agent {{researchAgent}}
             echo "::endgroup::"
-          done <<< "$GROUPS"
+          done
 
       - name: Review researched items (agent={{reviewAgent}})
         # Cross-agent review (ADR-0001): the review step intentionally uses
@@ -109,25 +129,4 @@ jobs:
               -d "{\"text\":\"feedradar: $UNSURE items need human triage review\"}"
           fi
 
-      - name: Create PR with research output
-        # `peter-evans/create-pull-request@v6` stages items/ state/ research/
-        # into a single PR per cron tick. Human reviews the PR before
-        # research/ lands on main, giving an explicit gate on auto-generated
-        # content (ADR-0014 §X5 / ADR-0018 §W5).
-        uses: peter-evans/create-pull-request@v6
-        with:
-          commit-message: "chore(feedradar): daily watch + triage + research"
-          # peter-evans/create-pull-request does not run shell on these
-          # fields, so `$(date ...)` would land literally in the PR title.
-          # Use the `github.run_id` expression to keep titles unique per run.
-          title: "feedradar: daily triage + research (run ${{ github.run_id }})"
-          body: |
-            Automated feedradar pipeline output. Review the research/ Markdown
-            before merging — generated content is untrusted (ADR-0009).
-          branch: feedradar/daily
-          base: ${{ github.ref_name }}
-          delete-branch: true
-          add-paths: |
-            items/
-            state/
-            research/
+{{finalStep}}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ozzylabs/feedradar",
-  "version": "0.1.7",
+  "version": "0.1.9",
   "description": "Multi-agent CLI that watches blogs and release feeds, then turns keyword hits into Markdown research reports",
   "type": "module",
   "publishConfig": {