Merged
Conversation
…te read/update paths - Add require_column() validation helper on Database::Impl - Add is_safe_identifier() character validation for 4 PRAGMA queries in schema.cpp - Add require_column calls to all 6 scalar read methods - Add require_column calls to all 6 vector read methods - Add require_column calls to all 6 set read methods - Add require_column calls to all 6 vector update methods - Add require_column calls to all 6 set update methods Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ites - Add column validation to update_time_series_files for caller-provided column names - Confirm database_create.cpp validated by TypeValidator - Confirm database_relations.cpp validated by FK lookup - Confirm database_delete.cpp validated by require_collection Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add ReadScalarIntegersInvalidColumnThrows test with error message verification - Add ReadVectorIntegersInvalidColumnThrows test with error message verification - Add UpdateVectorIntegersInvalidColumnThrows test with error message verification Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Create .clang-tidy with bugprone-*, modernize-*, performance-*, readability-identifier-naming checks - Disable noisy checks: easily-swappable-parameters, trailing-return-type, avoid-c-arrays, narrowing-conversions, use-nodiscard - Configure naming rules for CamelCase classes/structs/enums and lower_case functions/variables - Add CMake tidy target that reuses shared ALL_SOURCE_FILES glob - Fix glob to use src/c/*.cpp instead of src/c/*.c (no .c files exist) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…erification Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #79 +/- ##
==========================================
- Coverage 86.13% 86.06% -0.07%
==========================================
Files 92 92
Lines 10264 10344 +80
Branches 537 549 +12
==========================================
+ Hits 8841 8903 +62
- Misses 1398 1416 +18
Partials 25 25
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
- Use starts_with/ends_with (C++20) instead of substr/compare for prefix/suffix checks - Use emplace_back instead of push_back for variant types - Use auto with static_cast to avoid type duplication - Use designated initializers for aggregate types (TableMatch) - Remove redundant member initializers and return statements - Pass by value and use std::move for Migration constructor - Take sol::table by const ref in static helper functions - Add NOLINT for SQLite SQLITE_TRANSIENT macro (int-to-ptr cast) - Add NOLINT for intentional bugprone-branch-clone in value_to_string - Add NOLINT for unchecked-optional-access where check is done earlier in control flow - Add NOLINTBEGIN/NOLINTEND for sol2 lambda bindings (pass-by-value required by sol2) - Disable noisy checks globally: inefficient-string-concatenation, use-ranges, use-designated-initializers, enum-size - Add reserve() for vectors populated in loops - Use std::ranges::all_of for is_safe_identifier Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Transformation rules: C++ to C API, Julia, Dart, Lua naming patterns - Representative cross-layer examples table covering 14 operation categories - Binding-only convenience methods: DateTime wrappers, composite readers, multi-column group readers Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- SUMMARY.md with test results: 1213 tests across 4 suites all passing - STATE.md updated to 100% complete (15/15 plans, all 10 phases) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Archive milestone artifacts, evolve PROJECT.md, reorganize ROADMAP.md. - milestones/v1.0-ROADMAP.md: full phase details archived - milestones/v1.0-REQUIREMENTS.md: all 25 requirements marked complete - REQUIREMENTS.md deleted (fresh for next milestone) - PROJECT.md evolved with validated requirements and key decision outcomes - ROADMAP.md collapsed to milestone summary Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.