Skip to content

feat(docsync): docsync-get: CLI to get docsyncs#227

Merged
robinanterior merged 3 commits into
masterfrom
feat/docsync-get
Jun 16, 2026
Merged

feat(docsync): docsync-get: CLI to get docsyncs#227
robinanterior merged 3 commits into
masterfrom
feat/docsync-get

Conversation

@robinanterior

@robinanterior robinanterior commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Example:

$ docsync-get typescript/src | jq
{
  "Call": "A full brrr request payload. This is a low-level brrr primitive. The memo key must be generated by the instantiator of this class, and it must be deterministic: the \"same\" args and kwargs must always encode to the same memo key. Using the same memo key, we store the task and its argv here so we can retrieve them in workers.",
  "PendingReturns": "Set of parents waiting for a child call to complete. When the child call is scheduled, a timestamp is added to this record to indicate it doesn't need to be rescheduled. If the record exists but with a null scheduled timestamp, you cannot be sure this child has ever actually been scheduled, so it should be rescheduled. This record is used in highly race sensitive context and is the point of a lot of CASing."
}

@robinanterior robinanterior force-pushed the feat/docsync-get branch 2 times, most recently from 5fb78c2 to 5e76f74 Compare June 12, 2026 23:54
Comment thread docsync/src/cmds.ts Outdated
Comment thread docsync/src/cmds.ts Outdated
Comment thread docsync/src/cmds.ts Outdated
Comment thread docsync/package.nix Outdated
Comment thread docsync/package.nix Outdated
Base automatically changed from refactor/docsync to master June 15, 2026 19:10
@robinanterior robinanterior force-pushed the feat/docsync-get branch 2 times, most recently from d01286f to 503a97c Compare June 15, 2026 19:15

@the9000 the9000 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.

I'm approving this, assuming that you will explain the purpose and use of this code better.
For instance, it could work for pure Python modules in Python stdlib:

docsync-get $(python -c "import base64; print(base64.__file__)") b64encode

Comment thread docsync/src/cmds.ts
@robinanterior robinanterior force-pushed the feat/docsync-get branch 2 times, most recently from 18e8b88 to 2b5970b Compare June 15, 2026 21:30
@robinanterior

robinanterior commented Jun 15, 2026

Copy link
Copy Markdown
Contributor Author

I'm approving this, assuming that you will explain the purpose and use of this code better. For instance, it could work for pure Python modules in Python stdlib:

docsync-get $(python -c "import base64; print(base64.__file__)") b64encode

discussed offline: misunderstanding arose from my use of the implementation filename in the example docstring. Changed, to avoid this happening again. Ftr it doesn't take code as input, only source files.


sorry this was a different quote--the misunderstanding here was the purpose of the tool , operating only on explicitly defined comment tags, not generic code.

@robinanterior robinanterior added this pull request to the merge queue Jun 16, 2026
Merged via the queue into master with commit 5acfd4d Jun 16, 2026
4 checks passed
@robinanterior robinanterior deleted the feat/docsync-get branch June 16, 2026 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants