Skip to content

Merge stable into develop#9979

Closed
polmichel wants to merge 6 commits into
developfrom
pmi-merge-stable-into-develop
Closed

Merge stable into develop#9979
polmichel wants to merge 6 commits into
developfrom
pmi-merge-stable-into-develop

Conversation

@polmichel

@polmichel polmichel commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Supersedes #9976, which was blocked on merge conflicts.

Summary

Merges stable into develop.

Brings stable's release-notes docs redesign (auto-generated sidebar via docs/sidebar-releases.ts, a chronological feed at /release-notes/infrahub, required frontmatter on all release files validated by scripts/backfill-release-frontmatter.mjs --check), the cAdvisor image bump (ghcr.io/google/cadvisor:v0.60.5), and the actions/setup-python/actions/setup-node v7 bumps into develop.

Conflicts resolved

  • .agents/commands/cut-release.md — stable and develop had diverged on this command doc: stable carried the newer release-notes redesign (frontmatter contract + auto-generated sidebar) but the older pyproject.toml-bump versioning; develop carried the git-tag/hatch-vcs versioning model but a stale manual "Update Sidebar" step. Resolved by keeping develop's git-tag versioning throughout and adopting stable's auto-generated-sidebar section, then removing the now-dead docs/sidebars.ts references from the release steps (git add in the commit step and the verification checklist). Grounded in the merged tree, which auto-generates the sidebar (docs/sidebar-releases.ts present; docs/sidebars.ts lists no release entries). (1db87c1e1)

Conflicts raised for review

None requiring a decision. Two notes for the reviewer:

  • The cut-release.md resolution went slightly beyond the literal conflict markers to keep the doc internally consistent with the merged tree's generated-sidebar model (Steps 7–8 no longer stage/verify docs/sidebars.ts).
  • Pre-existing (not introduced by this merge): the Error Handling bullet "If the sidebar update location cannot be found…" still assumes the old manual-sidebar model. It is identical on both stable and develop; left untouched here to keep the merge focused — worth a follow-up tidy.

Versioning

No version-pin conflict arose. pyproject.toml / python_testcontainers/pyproject.toml kept develop's dynamic (hatch-vcs) versioning; the static fallback-version sentinel is identical on both branches by design, so the usual stable→develop version conflict did not occur. uv.lock merged cleanly.

Validation

  • node scripts/backfill-release-frontmatter.mjs --check (from docs/) — passes for all 115 release files
  • All 46 changed workflow / observability-compose YAML files parse cleanly
  • actions/setup-python / actions/setup-node consistently on v7 (the two remaining @v6 refs are gh-aw .md workflow definitions, v6 on both branches — not a merge artifact)

Backend unit/integration tests and docs.validate were not run: the merge changed no backend source, config, schema, events, or CLI files, so no backend generated files can be stale.


Summary by cubic

Merge stable into develop to bring the release-notes docs redesign and CI/observability updates. Also aligns the release process doc with the generated sidebar while keeping develop’s git‑tag versioning.

  • New Features

    • Release notes redesign: auto-generated sidebar via docs/sidebar-releases.ts and a chronological feed at /release-notes/infrahub. All release files now require release_date, release_type, and description (optional breaking) with validation via scripts/backfill-release-frontmatter.mjs --check. Updated cut-release.md to remove manual docs/sidebars.ts edits and match the generated-sidebar model.
  • Dependencies

    • Upgrade actions/setup-python to v7 and actions/setup-node to v7 across workflows.
    • Bump cAdvisor to ghcr.io/google/cadvisor:v0.60.5 in the observability docker-compose files.

Written for commit 1db87c1. Summary will update on new commits.

Review in cubic

Infrahub and others added 6 commits July 20, 2026 13:22
Bump cAdvisor from gcr.io/cadvisor/cadvisor:v0.52.1 to
ghcr.io/google/cadvisor:v0.60.5 in the observability docker-compose
stack (standalone and standard variants).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
* docs: redesign release-notes navigation and landing feed

Replace the flat 114-entry release-notes sidebar with one collapsible
category per release line ("1.10 release", "1.9 release", ...), generated
from the release files by docs/sidebar-releases.ts, and replace the
DocCard generated-index at /release-notes/infrahub with a chronological
release feed: full detail for the two newest lines, compact rows for
earlier ones.

- Add release_date / release_type / description (and breaking) frontmatter
  to all release files, with scripts/backfill-release-frontmatter.mjs to
  backfill and --check them
- Add docs/plugins/release-notes-data.js exposing release frontmatter as
  global data, consumed by the new <ReleaseFeed/> component
- Update the cut-release process docs for the new frontmatter contract
  and the generated sidebar

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* docs: fix mangled identifiers in release-note descriptions

The `description:` frontmatter added in 917a8de was drafted by
docs/scripts/backfill-release-frontmatter.mjs, whose stripMd() removed `_`
along with the markdown emphasis characters. Changelog entries quote their
identifiers in backticks, so both the backticks and the underscores were
stripped: `order_weight` became "orderweight", `sso_user_default_group`
became "ssouserdefaultgroup", and so on. <ReleaseFeed/> renders description
as plain text, so those mangled names were shown to readers verbatim. Vale
was correctly flagging them.

- Stop stripping `_` in stripMd() so future drafts keep snake_case intact
- Reword the 17 affected descriptions as prose. Four of them referenced CLI
  commands (`infrahub db check`, `infrahub upgrade`); the lowercase there was
  correct, so they are rephrased rather than capitalized
- Whitelist Starlette, backfills and upserts, which are correct as written

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

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Bumps [actions/setup-python](https://github.com/actions/setup-python) from 6 to 7.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](actions/setup-python@v6...v7)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [actions/setup-node](https://github.com/actions/setup-node) from 6 to 7.
- [Release notes](https://github.com/actions/setup-node/releases)
- [Commits](actions/setup-node@v6...v7)

---
updated-dependencies:
- dependency-name: actions/setup-node
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
…to-develop

# Conflicts:
#	.agents/commands/cut-release.md
Adopt the auto-generated release-notes sidebar (docs/sidebar-releases.ts)
from stable while keeping develop's git-tag-based versioning. Drop the
stale docs/sidebars.ts staging and verification references, since a
release no longer edits that file.
@github-actions github-actions Bot added type/documentation Improvements or additions to documentation group/ci Issue related to the CI pipeline labels Jul 20, 2026
@polmichel polmichel closed this Jul 20, 2026
@codspeed-hq

codspeed-hq Bot commented Jul 20, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 12 untouched benchmarks


Comparing pmi-merge-stable-into-develop (1db87c1) with develop (3f37924)

Open in CodSpeed

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

Labels

group/ci Issue related to the CI pipeline type/documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants