Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor /gen and /ted in %base; improve +help #7017

Open
wants to merge 17 commits into
base: develop
Choose a base branch
from

Conversation

sigilante
Copy link
Contributor

Pursuant to discussion in #7004.

  • /gen has been refactored into more consistent categories.
  • %story has been moved to its own repo.
  • /ted is planned to be similarly refactored.

@sigilante
Copy link
Contributor Author

  • /ted has been refactored to parallel /gen conventions.

@sigilante
Copy link
Contributor Author

sigilante commented Jun 13, 2024

+help now searches /ted as well as /gen; to wit,

> +help %code
+code     ::  Login code for web UI
-code     ::  Login code for web UI

We also support agent-directed generator notation:

> +helper %spider
:spider|kill        ::  Kill a thread
:spider|poke        ::  Poke a thread
:spider|start       ::  Start a thread
:spider|stop        ::  Stop a thread
+spider/tree        ::  List running threads

Now I need it to search for a buried one, and to handle paths. E.g.,

> +help %cp
|cp     ::  Copy a file in Clay

> +help /hood/cp
|cp     ::  Copy a file in Clay

I also intend to investigate /ted calling syntax in Dojo, as nested thread paths do not parse with /.

@sigilante
Copy link
Contributor Author

> +help /hood/gall
|gall/rein/hoon     ::  Adjust the state of agents on a desk.
|gall/sear/hoon     ::  Clear ship from pending queues
|gall/start/hoon    ::  Drum: boot unstarted app
|gall/verb/hoon     ::  Helm: Adjust Gall verbosity
|gall/sift/hoon     ::  Helm: Set Gall Verbosity by Agent
|gall/doff/hoon     ::  Kill old-style subscription.
|gall/goad/hoon     ::  Kiln: force Gall to rebuild agents
|gall/nuke/hoon     :: wipe agent state & subscriptions after confirmation

> +help /test
-test/hoon          <undocumented>

> +help %test
-test/hoon          <undocumented>

Looks like I'm missing passing in the file contents now.

@sigilante
Copy link
Contributor Author

sigilante commented Jun 14, 2024

> +help %code
+code     ::  Login code for web UI
|code     ::  Helm: query or reset login code for web
-code     ::  Login code for web UI

> +help %cp
|cp       ::  Kiln: copy file in clay
|clay/cp            ::  Kiln: copy file in clay

> +help /toot/sweet
/toot/sweet not found

> +help /hood/clay/cp
|clay/cp            ::  Kiln: copy file in clay

> +help %test
-test     ::  Run tests at paths.

> +help %toot
%toot not found

> +help %hood
|ames/close-flows   ::  Deletes all stale ames flows from failed (re) subscriptions
|commit   ::  Kiln: Commit from mount
* * *
|suspend            ::  Suspend the agents on a desk
|unmount            ::  Kiln: Remove unix mount

@sigilante sigilante changed the title Refactor /gen and /ted in %base Refactor /gen and /ted in %base; improve +help Jun 14, 2024
@sigilante sigilante marked this pull request as ready for review June 14, 2024 19:53
@sigilante
Copy link
Contributor Author

Oh, remaining desideratum: parse thread syntax like -azimuth/load in Dojo.

@sigilante
Copy link
Contributor Author

OK, got thread parsing with optional / supported now:

> -azimuth/load
ship: loading azimuth snapshot (137.928 points)
ship: processing azimuth snapshot (137.928 points)

> -azimuth-load
ship: loading azimuth snapshot (137.928 points)
ship: processing azimuth snapshot (137.928 points)

> -base!azimuth/load
ship: loading azimuth snapshot (137.928 points)
ship: processing azimuth snapshot (137.928 points)

> -base!azimuth-load
ship: loading azimuth snapshot (137.928 points)
ship: processing azimuth snapshot (137.928 points)

@sigilante
Copy link
Contributor Author

This is ready for review.

@sigilante
Copy link
Contributor Author

This will require some modest changes to documentation since a few things are moved. Important things are duplicated.

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.

None yet

1 participant