Skip to content

docs: restore OpenSpec logging task - #320

Merged
flowcool merged 1 commit into
mainfrom
clarify-operator-actions
Oct 27, 2026
Merged

flowcool merged 1 commit into
mainfrom
clarify-operator-actions

Conversation

@flowcool

Copy link
Copy Markdown
Owner

Behavior

Restore the missing OpenSpec task 3.2 and complete task 2.2 reason coverage. The OpenSpec apply instructions now expose all 10 implementation tasks, including the stdout/stderr and Supercronic acceptance.

Verification

  • openspec validate "clarify-operator-actions" --strict passes.
  • openspec instructions apply --change "clarify-operator-actions" --json reports tasks=10.

Docs-only correction. No runtime or production change.

@coderabbitai

coderabbitai Bot commented Oct 27, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Oct 27, 2026

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No new issues found.

Reviewed changes

I reviewed the one OpenSpec planning edit to openspec/changes/clarify-operator-actions/tasks.md. It contains no runtime or shipped code.

  • Task 2.2 reworded: it now lists signed-tax and reversal cases, keeps automatic retries and evidence-limited diagnostics separate, and requires one action/proof mapping per rejection reason. This matches the section 2 rule that already existed.
  • Task 3.2 restored: it covers the stdout/stderr split by level, Supercronic passthrough and the ghostfolio-ibkr-sync build <APP_VERSION> provenance line. The requirements it cites (Clear application log framing, Minimal native lab evidence) exist in the change's specs/connector-runtime/spec.md. The files it lists exist, except tests/test_logging.py, which the task is meant to create.

Pullfrog  | Fix it ➔ | View workflow run | Using claude-opus-5-5 | 𝕏

@flowcool
flowcool merged commit a074ef3 into main Oct 27, 2026
11 checks passed
@flowcool
flowcool deleted the clarify-operator-actions branch October 27, 2026 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant