Skip to content

feat: add [MANUAL] notation for human interaction steps#110

Closed
Bortlesboat wants to merge 1 commit intogoogleworkspace:mainfrom
Bortlesboat:feat/manual-notation
Closed

feat: add [MANUAL] notation for human interaction steps#110
Bortlesboat wants to merge 1 commit intogoogleworkspace:mainfrom
Bortlesboat:feat/manual-notation

Conversation

@Bortlesboat
Copy link
Contributor

@Bortlesboat Bortlesboat commented Mar 5, 2026

Summary

Changes

  • auth_commands.rs: login and setup commands marked [MANUAL] in help text; browser-open prompt prefixed with [MANUAL]
  • setup.rs: OAuth consent screen setup instructions prefixed with [MANUAL] for each manual step; browser login step marked
  • skills/gws-shared/SKILL.md: Auth section documents which methods are [MANUAL] vs automated

Design

Following the issue's spec, [MANUAL] is:

  • Simple and grep-able (grep -r '\[MANUAL\]')
  • Parseable by both humans and AI agents
  • Placed at the start of the relevant line/step

Phase 2 (structured JSON hint objects) is left for a follow-up PR.

Addresses #66

Test plan

  • [MANUAL] annotations are grep-able in help output
  • cargo test passes (CI will verify)
  • cargo clippy -- -D warnings clean (CI will verify)

@gemini-code-assist
Copy link
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@changeset-bot
Copy link

changeset-bot bot commented Mar 5, 2026

🦋 Changeset detected

Latest commit: f779fab

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@googleworkspace/cli Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@google-cla
Copy link

google-cla bot commented Mar 5, 2026

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@Bortlesboat
Copy link
Contributor Author

I have signed the Google CLA — please re-check.

@Bortlesboat Bortlesboat force-pushed the feat/manual-notation branch 7 times, most recently from fb62359 to 4a58bd0 Compare March 5, 2026 18:16
@Bortlesboat Bortlesboat force-pushed the feat/manual-notation branch from df3d39f to f779fab Compare March 5, 2026 18:21
@jpoehnelt jpoehnelt added area: docs cla: yes This human has signed the Contributor License Agreement. complexity: medium Moderate change, some review needed labels Mar 5, 2026
@jpoehnelt jpoehnelt closed this Mar 5, 2026
@codecov
Copy link

codecov bot commented Mar 6, 2026

Codecov Report

❌ Patch coverage is 28.57143% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.19%. Comparing base (c6095dd) to head (f779fab).
⚠️ Report is 27 commits behind head on main.

Files with missing lines Patch % Lines
src/error.rs 0.00% 3 Missing ⚠️
src/auth_commands.rs 66.66% 1 Missing ⚠️
src/setup.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #110   +/-   ##
=======================================
  Coverage   56.19%   56.19%           
=======================================
  Files          38       38           
  Lines       13860    13860           
=======================================
  Hits         7788     7788           
  Misses       6072     6072           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: docs cla: yes This human has signed the Contributor License Agreement. complexity: medium Moderate change, some review needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants