msvc analysis: suppress C26819/C28251 advisories, bump EOL actions

[borisbat]

Superseded by #3117 — this branch was accidentally based on bbatkin/style031-table-insert-lint instead of master, so it dragged in unrelated style031 files. Reopened clean as #3117.

[Copilot]

Pull request overview

This PR combines two themes: (1) keeping the manual MSVC /analyze SARIF workflow clean by suppressing advisory-only warnings and updating action versions, and (2) extending/repairing style_lint around table initialization patterns (including adding STYLE031) and updating docs/tests/tutorials accordingly.

Changes:

• Suppress MSVC /analyze advisories C26819 and C28251 in CustomRules.ruleset, and bump upload-sarif/upload-artifact action versions in the MSVC workflow.
• Fix STYLE021’s JsonValue type matching (struct in json module) and add STYLE031 (table insert/[]= run → table/set literal) to daslib/style_lint.das, with new lint tests.
• Update docs, tutorials, and tests to prefer table literals or to explicitly suppress STYLE021/STYLE031 where incremental insert patterns are intentional.

Reviewed changes

Copilot reviewed 21 out of 21 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
utils/lint/tests/style031_table_insert_run.das	New lint test coverage for STYLE031 behaviors and exclusions.
utils/lint/tests/style021_jv_table_insert.das	New lint test coverage for STYLE021 vs STYLE031 “division of labor”.
tutorials/language/47_data_walker.das	Uses table literal initialization and removes self-> method-call arrow usage.
tutorials/language/41_serialization.das	Uses table literal initialization for tutorial example.
tutorials/language/30_json.das	Adds nolint:STYLE021 to keep incremental JSON object-building example.
tutorials/language/10_tables.das	Adds nolint:STYLE031 to keep insert-based tutorial example.
tests/with_boost/test_with_table.das	Converts insert-run setup to a table literal.
tests/with_boost/test_with_lock_panics.das	Renames unused with_ binders and converts one table setup to a literal.
tests/table_packed/test_packed.das	Adds nolint:STYLE031 to keep insert/index-assign path under test.
tests/table_packed/test_packed_constkey.das	Adds nolint:STYLE031 where insert/index paths are the test subject.
tests/long_array_table/test_long_length.das	Converts simple insert-run setup to a table literal.
tests/long_array_table/test_fusion_table_i64.das	Adds nolint:STYLE031 to keep fused write path under test.
tests/long_array_table/test_dapi_layout.das	Adds nolint:STYLE031 to keep canonical insert path under test.
tests/linq/test_linq_table_source.das	Converts insert-run table factory to a table literal return.
tests/language/test_value_table_key.das	Adds nolint:STYLE031 and adjusts get(...) callback syntax usage.
skills/style_lint.md	Documents STYLE029/030/031 and corrects STYLE021 implementation notes.
doc/source/reference/language/lint.rst	Extends lint reference docs to include STYLE031 and STYLE021→STYLE031 fallback note.
daslib/style_lint.das	Implements STYLE031 and fixes STYLE021 JsonValue type matching.
CustomRules.ruleset	Suppresses C26819/C28251 advisories for MSVC analysis runs.
CLAUDE.md	Updates migration/style table with STYLE031 guidance.
.github/workflows/msvc.yml	Bumps upload-sarif to v3 and upload-artifact to v4.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.