chore(opencode): add ponytail plugin - #37
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 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. Comment |
…pers - Delete docs/setup.md pointer, root ps1/bat forwarders, opencode.json wrapper, social-card.svg - vite.config sourcemap default, package.json empty devDeps + check alias - Inline Mark/Icon/DownloadRow spans; protocol.md canonical for transport defaults - Net: -63 lines across 11 files
- Shared SDK/OBS+Qt setup out of android.yml/obs-plugin-windows.yml/release.yml - Release signing, artifact names, timeouts unchanged - Net workflows: +8/-76 plus 2 new actions
- New tests/_helpers.py: block_after + function_body + read_text - srt_* -> test_srt_contract, audio_* -> test_audio_contract, reservation/control/slot -> test_reservation_contract, video/transport -> test_video_transport_contract - One canonical release test; parsers kept with mirror note - 91 passed
…lpers - AvDeleter template; enum_ipv4_adapters; open_decoder; button table; forward_to_camera - Drop generation, last_gap_ns_, stopped_; shared contract_helpers.hpp; beacon keeps instanceId - Skipped: Qt json (OBS-core adapter rule), semver/strncasecmp/socket-send/Generic (false positives) - C++ tests + 91 pytest passed
- Formatter.formatIpAddress; CONTROL_PORT const; parseJson helper; LinkedHashMap LRU; bracketForUrl; codecConfigBytes; hasWideFocalSpread; setTorchUi - Drop release(), cancelCameraRecovery, normalizeHost, availabilityLabel/displayEndpoint, Fallback720p30, TelemetrySampler, PreferHevc, EncoderSelection - Kept: readAsciiLine (no over-buffer), double guard (race), fromDiscoveredDevice (contract-pinned) - Net -148 lines; needs CI for gradle (no SDK here)
…ical lists - ForwardingProtocol; wait_until; fold _require_ffmpeg_component; inline read_progress; removeprefix; ffmpeg_output_has_token; os.devnull; shared terminate; drop ReceiverConfig - build_plugin.bat :verify_sha256/:gen_lib; per-OS canonical stale lists; parameterized Test-OpenStreamInstaller - Kept probe flags (hw calibration knobs); 91 pytest passed
YashasVM
left a comment
YashasVM
left a comment
There was a problem hiding this comment.
Yashas's code review bot: This PR is not merge-ready at the current head. Android APK fails in the unit-test/lint/build step, and OBS Plugin Windows fails while building the plugin package; the Windows contract/behavioral tests, ABI verification, package checks, installer build, and smoke test are consequently skipped. Because this branch also contains substantial workflow refactoring and application/code changes beyond the stated ponytail-plugin intent, these failures need to be fixed and the full exact-head CI suite rerun successfully before merge.
1 participant
Adds opencode.json with @dietrichgebert/ponytail plugin.
Injects the ruleset every turn at the active level and adds /ponytail commands (lite/full/ultra/off). OpenCode auto-loads AGENTS.md, so repo rules hold even without the plugin.
Checkout alternative (not used): {"plugin": ["./.opencode/plugins/ponytail.mjs"]} — ./ resolves against opencode.json location; absolute path can share one checkout across projects.