Skip to content

[deep-report] Restore frontmatter anchor coverage for update-check and runtime flags#36961

Merged
pelikhan merged 5 commits into
mainfrom
copilot/deep-report-fix-check-for-updates-anchor
Jun 4, 2026
Merged

[deep-report] Restore frontmatter anchor coverage for update-check and runtime flags#36961
pelikhan merged 5 commits into
mainfrom
copilot/deep-report-fix-check-for-updates-anchor

Conversation

Copilot AI commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

frontmatter schema text linked to #check-for-updates, but the curated reference page had no matching heading. The same page also had zero curated coverage for run-install-scripts.

  • Curated frontmatter coverage additions

    • Added a dedicated check-for-updates section to docs/src/content/docs/reference/frontmatter.md so the schema link target now exists.
    • Added a minimal, user-facing run-install-scripts section with concise behavior notes.
  • Curated docs alignment with current fields

    • Removed the disable-model-invocation section from the curated reference page because that field has been removed.
  • Anchor + schema-link alignment

    • The heading format (### \check-for-updates`) creates the expected #check-for-updates` doc anchor used by schema descriptions.
  • Docs content style

    • Kept additions intentionally minimal and reference-oriented: one-paragraph semantics + small YAML example per field.
check-for-updates: true

runtimes:
  node:
    run-install-scripts: true

Copilot AI and others added 2 commits June 4, 2026 17:11
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix broken #check-for-updates doc anchor and add docs for three fields [deep-report] Restore frontmatter anchor coverage for update-check and model/runtime flags Jun 4, 2026
Copilot AI requested a review from pelikhan June 4, 2026 17:19

Keys must be 1–64 characters; values are string-only, up to 1024 characters.

### `disable-model-invocation`

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@copilot this field is removed and should be removed from the docs

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Removed from the curated frontmatter docs in commit e62eed4.

…erence

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [deep-report] Restore frontmatter anchor coverage for update-check and model/runtime flags [deep-report] Restore frontmatter anchor coverage for update-check and runtime flags Jun 4, 2026
Copilot AI requested a review from pelikhan June 4, 2026 17:39
@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Hey @Copilot 👋 — thanks for this documentation fix! Restoring the #check-for-updates anchor and adding the run-install-scripts section keeps the frontmatter reference properly aligned with the schema links — that's a solid housekeeping win.

One thing would help before this lands:

  • Split out the lock file changes.github/workflows/dead-code-remover.lock.yml contains a batch of hash-token rotations (heredoc delimiter strings like GH_AW_PROMPT_18306a05ecb4d558_EOFGH_AW_PROMPT_78a1d3ec5c7dcc5e_EOF) that are unrelated to the frontmatter docs work. These look like an incidental workflow regeneration. Keeping them in a separate commit or PR makes it easier for reviewers to confirm the docs change in isolation.

If you would like a hand tidying this up:

In PR branch `copilot/deep-report-fix-check-for-updates-anchor`, the file `.github/workflows/dead-code-remover.lock.yml` contains hash-token rotations (heredoc delimiter renames) that are unrelated to the frontmatter documentation changes in `docs/src/content/docs/reference/frontmatter.md`.

Please:
1. Revert the changes to `.github/workflows/dead-code-remover.lock.yml` in this branch so the PR contains only the frontmatter documentation additions.
2. If the lock file regeneration is genuinely needed, open it as a separate PR with a clear explanation of why the hashes changed.

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • patchdiff.githubusercontent.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "patchdiff.githubusercontent.com"

See Network Configuration for more information.

Generated by ✅ Contribution Check · sonnet46 4.2M ·

@pelikhan pelikhan marked this pull request as ready for review June 4, 2026 18:01
Copilot AI review requested due to automatic review settings June 4, 2026 18:01
@pelikhan pelikhan merged commit f344d1e into main Jun 4, 2026
2 checks passed
@pelikhan pelikhan deleted the copilot/deep-report-fix-check-for-updates-anchor branch June 4, 2026 18:01

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot stopped reviewing on behalf of pelikhan due to an error June 4, 2026 19:01
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.

[deep-report] Fix broken #check-for-updates doc anchor and add docs for 3 zero-coverage frontmatter fields

3 participants