Skip to content

feat(article): add identity-deep-dive radar intro - #13

Merged
kanywst merged 3 commits into
mainfrom
article/identity-deep-dive-radar
Oct 18, 2026
Merged

kanywst merged 3 commits into
mainfrom
article/identity-deep-dive-radar

Conversation

@kanywst

@kanywst kanywst commented Oct 13, 2026

Copy link
Copy Markdown
Member

Intro article for 0-draft/identity-deep-dive, written as the exposure step from the 2026-10-13 archive check (kanywst/bellman#307).

  • published: false with date: 2026-11-13T00:00:00Z, so merging schedules it: schedule.yml flips it live at that time. Change or drop the date before merging if you want it held.
  • The numbers and the scoring excerpt come from the 2026-10-13 daily run (candidates.json, reports/daily/2026-10-13.md) and tracks/oauth-wg/config/scoring.yaml.
  • markdownlint-cli2 passes locally with the repo config. validate_articles.py could not run locally, so CI is the check for frontmatter and links.

@coderabbitai

coderabbitai Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f63bb6cb-6ca2-4c13-9ba4-6275007120b6
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Comment thread articles/identity-deep-dive-radar.md
Comment thread articles/identity-deep-dive-radar.md Outdated
@claude

claude Bot commented Oct 13, 2026

Copy link
Copy Markdown

Only articles/identity-deep-dive-radar.md changed. Frontmatter fits the repo rules: 4 tags, a future date, and published: false, so the hourly schedule.yml will flip it. The WIMSE deep-dive link resolves to a real article in INDEX.md.

Two things need a source check before publishing (inline comments posted):

  • Line 89: the expansion "AI Identity Management System" for draft-ietf-wimse-aims-00 reads as recalled. The run figures at lines 72 and 88-90 are also unverifiable from this repo.
  • Line 94: "the first one covers draft-ietf-oauth-v2-1" has no link.

Line 16 says "more than sixty GitHub repositories" while line 90 reports 65 OpenID repos alone. Make sure the totals are consistent across all tracks.

Comment thread articles/identity-deep-dive-radar.md Outdated
Comment thread articles/identity-deep-dive-radar.md Outdated
@claude

claude Bot commented Oct 13, 2026

Copy link
Copy Markdown

Reviewed the new articles/identity-deep-dive-radar.md. Frontmatter fits repo conventions (published: false with a future UTC date, existing ShowDev series, 4 tags). The WIMSE deep-dive link matches INDEX.md.

Two items to check before the scheduled publish:

  • Lines 86-90: the daily-run figures are a 2026-10-13 snapshot but the article publishes 2026-11-13, and the draft-ietf-wimse-aims-00 title is unsourced.
  • Lines 94 and 98: the cross-repo links (deep-dives/draft-ietf-oauth-v2-1, templates/new-track/) were not verified.

No EN/JA parity or workflow/injection issues in this diff.

@claude

claude Bot commented Oct 18, 2026

Copy link
Copy Markdown

No blocking findings. The article follows the repo's draft convention (published: false with a future UTC date), and the WIMSE deep-dive link matches an existing article in INDEX.md.

Not verified, because the sandbox refused my gh api calls:

  • Link visibility: dd967ed just removed links to repos that went private. Confirm 0-draft/identity-deep-dive, its Pages site, and the deep-dives/draft-ietf-oauth-v2-1 tree path are public before the scheduled publish on 2026-11-13.
  • Snapshot numbers: the 2026-10-13 figures (65/20/6 repos, 172 PRs, 1,521 issues, the scores, and the draft-ietf-wimse-aims-00 title) have no source in the diff. The article already labels them a point-in-time snapshot.
  • YAML excerpt: compare the scoring excerpt against the real scoring.yaml so the "abridged" rubric is accurate.
  • Intro wording: the intro says "on the day I checked" without a date, unlike the later 2026-10-13 snapshot. Consider stating the same date in both places.

There is no JA counterpart in the repo (JA copies are gitignored), so there is no parity issue.

@kanywst
kanywst merged commit 246ac01 into main Oct 18, 2026
3 checks passed
@kanywst
kanywst deleted the article/identity-deep-dive-radar branch October 18, 2026 01:52
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.

1 participant