Add Local Delegated Agent Identity Architecture draft#1
Conversation
| gem install kramdown-rfc | ||
| python -m pip install xml2rfc |
There was a problem hiding this comment.
Let's pin these dependencies to the known good versions.
| pages: write | ||
| id-token: write |
There was a problem hiding this comment.
Small nit but it could be possible to scope the write permissions directly onto the deploy job (link) rather than the whole workflow.
| if not all([parent_refs_block, norm_block, inf_block, ack_block, about_block]): | ||
| sys.exit(0) |
There was a problem hiding this comment.
What do you think about failing loudly here, if an expected block isn't found?
There was a problem hiding this comment.
Yeah good point but I think we maybe should leave atm and log something in the backlog to look more deeply into this reorder script. It hard-codes assumptions about the current draft's structure (references at s13, the 13.1/13.2 numbering, ack blocks), so future drafts may not match. I think we should leave as is for now and I'll write up a follow-up ticket.
| lint: $(TXT) | ||
| @for f in $(TXT); do idnits --verbose $$f; done |
There was a problem hiding this comment.
Would it be useful to make lint in CI as well?
2 participants
Adds the first published IETF Internet-Draft in 1Password's Agent Identity series, plus the repo scaffolding to render it as HTML via GitHub Pages.
What's included
draft-1password-agent-identity-local-delegated.md-> thedraft-1password-agent-identity-local-delegated-00(kramdown-rfc)..github/workflows/build.ymlrender the draft to .html/.txt/.xml (kramdown-rfc + xml2rfc) and deploy to GitHub Pages.scripts/gen-index.shgenerates a landing page by globbingdraft-*.html.assets/1password.css, logo/icon SVGs, and the four local-delegated architecture diagrams.Notes for reviewers