Skip to content

Reconcile enterprise/cloud doc divergence + add bump-last-updated tooling#1547

Open
justinegeffen wants to merge 30 commits into
masterfrom
enterprise-cloud-divergence-audit
Open

Reconcile enterprise/cloud doc divergence + add bump-last-updated tooling#1547
justinegeffen wants to merge 30 commits into
masterfrom
enterprise-cloud-divergence-audit

Conversation

@justinegeffen

Copy link
Copy Markdown
Contributor

Summary

  • Fixes drift bugs surfaced by an enterprise vs cloud doc audit (broken refs, half-completed rebrands, regressions, UI-label inconsistencies, stale Bitbucket flow, etc.)
  • Normalizes frontmatter date format across both trees to date created: "YYYY-MM-DD" + last updated: "YYYY-MM-DD"
  • Adds a bump-last-updated pre-commit hook + extends pre-commit-fix.yaml to auto-trigger on PRs so last updated: stays current without manual effort

The full per-file rationale (decisions, ports, audit corrections, deliberately-untouched items) lives in enterprise-cloud-divergence-audit.md. That doc is the canonical decision log — it deliberately keeps this PR description short.

Test plan

  • Confirm the new bump-last-updated hook works locally (pre-commit run bump-last-updated --all-files)
  • Confirm the extended pre-commit-fix.yaml workflow runs on this PR's synchronize event and that the bot only commits a bump if dates are stale
  • Spot-check a few of the user-direction calls (R-IDE rebrand, "Revision" UI label, studio:execute Launch=❌, j21-26.04 launcher row)
  • Verify the Antigravity skill port renders correctly on the preview build
  • Verify the Bitbucket section update on Enterprise renders correctly

🤖 Generated with Claude Code

…ling

Fixes drift bugs surfaced by the enterprise vs cloud audit, normalizes
frontmatter date format across both trees, and adds a pre-commit hook +
CI auto-trigger so last updated dates stay current. Full per-file
rationale in enterprise-cloud-divergence-audit.md.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@netlify

netlify Bot commented Jun 8, 2026

Copy link
Copy Markdown

Deploy Preview for seqera-docs ready!

Name Link
🔨 Latest commit a370e98
🔍 Latest deploy log https://app.netlify.com/projects/seqera-docs/deploys/6a4c11b5f744a80008897499
😎 Deploy Preview https://deploy-preview-1547--seqera-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@justinegeffen justinegeffen added 1. Editor review Needs a language review 1. Dev/PM/SME Needs a review by a Dev/PM/SME do not merge Do not merge until this label is removed claude-code-assisted Vibe-coded but with human oversight and guidance. Must be validated by another human and co-pilot. content-improvement This work improves content editorially or structurally. labels Jun 8, 2026

@christopher-hakkaart christopher-hakkaart left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most of this PR is cosmetic and date added and changed improvements. So it looks worse than it actually is.

95% of these changes are not offensive.

I've been through the changes and flagged anything that I think needs checked. IMO - everything else can be skimmed.

The report at the end flags some important differences and can be reviewed separately for quick wins.

Comment thread .github/scripts/bump-last-updated.py
Comment thread platform-enterprise_docs/co-scientist/skill-antigravity.md
Comment thread platform-enterprise_docs/functionality_matrix/overview.md
Comment thread platform-enterprise_docs/git/overview.md
Comment thread platform-enterprise_docs/orgs-and-teams/roles.md
Comment thread platform-enterprise_docs/pipelines/versioning.md Outdated
Comment thread platform-enterprise_docs/secrets/overview.md Outdated
justinegeffen and others added 11 commits June 9, 2026 21:57
Co-authored-by: Chris Hakkaart <chris.hakkaart@seqera.io>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Co-authored-by: Chris Hakkaart <chris.hakkaart@seqera.io>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Per PR review (christopher-hakkaart): pre-commit hook now fails explicitly
with the exact fix command, matching the existing check-doc-tags pattern.
CI workflow still auto-bumps on PR via a dedicated step so fork
contributors and skipped local hooks merge with current dates.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
@justinegeffen justinegeffen removed the do not merge Do not merge until this label is removed label Jun 12, 2026
@christopher-hakkaart

Copy link
Copy Markdown
Member

I'll check the scripts again next week. But otherwise looking good.

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Seqera Docs Bot and others added 3 commits June 18, 2026 15:04
Comment thread .github/workflows/pre-commit-fix.yaml
@christopher-hakkaart

christopher-hakkaart commented Jun 29, 2026

Copy link
Copy Markdown
Member

This is a massive PR. I worry it could get stuck in an endless review loop if we try to address everything in the audit file. I suggest we verify the changes made as a part of this PR, and move everything else to new, smaller PRs.

Editorially, everything looks okay to me. Vale and other checks might light up as so many files are being touched.

Enterprise ↔ Cloud divergence — content verification checklist

Verify each applied change against the live product. Tick once confirmed correct (or flag if it needs reverting).

Content ports — confirm the ported content is accurate for its destination

  • Bitbucket API tokensplatform-enterprise_docs/git/overview.md: confirm the API-tokens flow is correct for Enterprise's supported Bitbucket integration — Username = account email, Password field renamed to Token, and the deprecation dates.
  • Antigravity / Gemini skillplatform-enterprise_docs/co-scientist/skill-antigravity.md (new file) + Antigravity bullet in co-scientist/coding-agents.md: confirm the Antigravity/Gemini integration actually works on Enterprise deployments, not just Cloud.
  • AWS Secrets Manager ListSecrets warningplatform-enterprise_docs/secrets/overview.md: confirm the IAM gotcha (scoping the policy down without keeping ListSecrets on * → secret-deletion failures / stuck workflows) genuinely applies to Enterprise.
  • Launch form fieldsplatform-enterprise_docs/getting-started/quickstart-demo/launch-pipelines.md: confirm Version name, Version ID, Main script, and Schema (Repository default / Repository path / Seqera Platform schema) all appear in the Enterprise launch form.
  • /migrate-from-wdl and /write-nf-test skillsplatform-cloud/docs/co-scientist/reference/skills-reference.md: confirm both skills exist and are available on Cloud.

Product-fact decisions — confirm the value matches the live product

  • studio:execute for Launch roleplatform-enterprise_docs/orgs-and-teams/roles.md: confirm the Launch role's studio:execute is actually False (now ❌ on both trees).
  • Default launcher j21-26.04platform-enterprise_docs/functionality_matrix/overview.md, 26.1.0 row: confirm the default launcher from 26.1.0 is j21-26.04 (new format) on both Cloud and Enterprise.
  • Token placeholder <ACCESS_TOKEN>credentials/agent_credentials.md (both trees), getting-started/quickstart-demo/automation.md (Enterprise), supported_software/agent/overview.md (both trees): confirm <ACCESS_TOKEN> is the right placeholder and doesn't contradict any literal token name the CLI/API actually requires.

versioning.md reconciliation — confirm the ported content

  • Complete JSON params exampleplatform-enterprise_docs/pipelines/versioning.md: confirm the full "my_parameter": { ... } example (which replaced the old truncated { "track_changes": false } fragment) is correct and valid.
  • Deletion-scope wordingplatform-enterprise_docs/pipelines/versioning.md: confirm "Individual draft versions cannot be deleted" is accurate — i.e., that non-draft versions behave differently.

@justinegeffen

Copy link
Copy Markdown
Contributor Author

@copilot resolve the merge conflicts in this pull request

Copilot AI commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

The merge conflicts were already resolved in commit 89536a1.

The branch also had a failing pre-commit check due to three files with stale last updated: dates after the master merge — fixed in the latest commit by bumping them to today (2026-07-06):

  • platform-cloud/docs/compute-envs/preflight-checks.mdx
  • platform-cloud/docs/getting-started/quickstart-demo/add-data.md
  • platform-cloud/docs/getting-started/deployment-options.md

The Lint prose failure is a GitHub API size limit — the PR touches >300 files, so reviewdog can't fetch the diff. That's a known limitation of large PRs and isn't blocking.

@justinegeffen

Copy link
Copy Markdown
Contributor Author

@copilot resolve the merge conflicts in this pull request

Copilot AI commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Resolved in commit a370e98. Two new master commits (#1632, #1633) had landed since the last merge, causing conflicts in three files. Resolved by taking master's updated content and keeping the normalized frontmatter date format from the branch:

  • platform-cloud/docs/data-privacy/overview.md
  • platform-enterprise_docs/data-privacy/overview.md
  • platform-enterprise_docs/limits/overview.md

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

Labels

1. Dev/PM/SME Needs a review by a Dev/PM/SME 1. Editor review Needs a language review claude-code-assisted Vibe-coded but with human oversight and guidance. Must be validated by another human and co-pilot. content-improvement This work improves content editorially or structurally. deslop-to-do

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants