Update elasticsearch#10
Open
ttys0dev wants to merge 777 commits into
Open
Conversation
ttys0dev
force-pushed
the
update-elasticsearch
branch
2 times, most recently
from
1b42bc8 to
aa15a61
Compare
Minimize old bot comments as OUTDATED and post a fresh markdown summary when warnings or errors are found. Silent when clean. Also adds concurrency group and job summary output. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add back docket foreign keys on `CaseTransfer` model and add constraint that at most one can be null.
Legacy templates style links by default in override.css so the warnings are mostly irrelevant. To prevent noise we now only check styled links in new templates.
Co-authored-by: Elisa <elisa@free.law>
Now that BLOCK_UNKNOWN_FILTERS has been enabled in production since Feb 9, the Redis logging and email notification system for bad API filter parameters is dead code. Invalid requests are simply rejected with HTTP 400. Remove: - Redis logging functions (log_bad_filter_params_to_redis, etc.) - BAD_FILTER_PARAMS_PREFIX constant - notify_bad_api_filter_users management command - bad_api_filter_params_email.txt template - BLOCK_UNKNOWN_FILTERS setting (blocking is now always on) - _handle_unknown_params method (inlined into filter_queryset) Fixes: freelawproject#6771 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove @override_settings(BLOCK_UNKNOWN_FILTERS=...) from all test classes since blocking is now the default behavior - Replace UnknownFilterParameterTests (Redis-based) with UnknownFilterParameterBlockingTests (HTTP integration tests) - Remove test_can_ignore_invalid_filters (non-blocking mode removed) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
add xml_scan to bulk data script update migrations
…as-and-scotus-model-changes Update `SCOTUSDocument`, `TexasDocument`, and `CaseTransfer` models
…bankruptcy feat(help): Add new bankruptcy info to v2 template as well
…odcast-icons-20260217 Use correct podcast icon
…xtures_20260130 test(recap): replace fixture with factory
add new migration to add a new field docket_number_source to the search app
This commit introduces `MAX_RELATED_SEARCH_PAGINATION_DEPTH` (default: 5 pages) and updates `check_pagination_depth()` to accept a configurable max_depth parameter.
This commit adds logic to identify `related:` queries in `fetch_and_paginate_results()` and apply the stricter `MAX_RELATED_SEARCH_PAGINATION_DEPTH` limit to prevent crawlers from triggering expensive deep-pagination MLT queries.
for more information, see https://pre-commit.ci
The check_pdf_redactions task references httpx exception types but httpx was not imported, causing a NameError at module load time. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ray-redaction-checking feat(recap): Add X-Ray redaction checking to PDF ingestion pipeline
…e-ein-secret-20260311 Remove EIN_SECRET
…crapers-tests-20260204 test(corpus_importer,scrapers): replace fixtures with factories
…change-log feat(help): Apply updated design to REST API Change Log
for more information, see https://pre-commit.ci
The legacy date-based Redis key (e.g. search.results.2026-03-16) has no per-test-class prefix, so integration tests running in parallel can increment it between setUp cleanup and assertion. Use delta-based checks (consistent with PrometheusIntegration*Tests) instead of absolute values. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…al-court-data Add `TrialCourtData` Model
The <a> opening tag was excluded from the depth count but </a> was still counted, creating a depth imbalance that prevented the scanner from reaching the actual ancestor element. Since nested <a> tags are invalid HTML, there is no reason to special-case them — count all tags equally. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…duce-prometheus feat(stats): Add Prometheus metrics for monitoring
These template changes belong in the dependent PR that uses the ancestor styling pattern — this PR should only contain the lint check fix. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Links that wrap images or SVGs don't need text-styling classes.
Skip the warning when the <a> tag's content is only <img>, <svg>,
or {% svg %} template tags.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add a docs link to the _has_ancestor_link_styling docstring and a "Help: Unstyled links" section to the markdown summary that appears in PR comments when the check fails. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Adds a comment to touch the file so the CI check runs on it. Remove before merge. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…inks-ancestor-styling fix: improve bare-link check for ancestor styling and visual content
…ject#6907) * feat(ai): add GenAI batch wrapper and management commands * refactor(ai): extract helper functions from check_gemini_batch_status command * refactor(ai): extract helpers from send_gemini_batches, fix TaskStatus in both batch commands * fix(ai): add missing init files * fix(ai): use .count() to count pending requests logger.error instead of logger.warning when cant serialize json delete temp files in fetch_files_and_create_tasks if something fails update tests * fix(ai): use LLMRequestStatusChoices class in LLMRequest objects and queries update COMPLETED_STATES set set failed_tasks count in LLMRequest * fix(ai): handle transient errors in batch status check * refactor(ai): wrap batch creation in atomic transaction - use transaction.atomic() so LLMRequest and tasks roll back on failure instead of leaving partial DB state - pass temp_files as mutable param so cleanup works even on partial failure - update test * refactor(ai): replace LLMTaskStatusChoices with LLMRequestStatusChoices in tests and in send_gemini_batches command * fix(ai): use JobState enums and handle PARTIALLY_SUCCEEDED - Compare job.state (enum) instead of job.state.name (string) against COMPLETED_STATES and SUCCEEDED_STATES sets - Add SUCCEEDED_STATES set for states where results can be downloaded - Handle JOB_STATE_PARTIALLY_SUCCEEDED by processing available results - Mark orphaned IN_PROGRESS tasks as FAILED after processing results - Use .count() variable to avoid evaluating pending_requests queryset twice * feat(ai): rename command and add batch size control - Rename send_gemini_batches to send_gemini_file_batches - Add --batch-size argument (default 1000) to split large S3 paths into multiple LLMRequests - Add chunk_iterator() helper for splitting iterators - Rename fetch_files_and_create_tasks to create_tasks_from_files, accepting file tuples directly - Wrap all chunks in a single transaction * test(ai): update tests for enum and batch splitting changes - Use JobState enum values instead of string mocks in all tests - Use LLMRequestStatusChoices for LLMRequest assertions - Update test_check_status_task_not_found for orphaned task cleanup - Add test_send_gemini_batches_splits_by_batch_size - Update mock references for send_gemini_file_batches rename * fix(ai): address PR review feedback from grossir - Accept PARTIALLY_SUCCEEDED in download_results via DOWNLOADABLE_JOB_STATES set - Use bulk update() in process_failed_request instead of per-task loop - Mark tasks as failed in transient-error-expiry and generic exception paths - Move transaction.atomic inside chunk loop for per-batch rollback - Add outer except for S3 iteration failures with Sentry reporting - Update tests for new behavior * refactor(ai): move model validation into GoogleGenAIBatchWrapper - Add SUPPORTED_GEMINI_MODELS dict with shutdown dates to google.py - Add validate_model() static method with expiration checking - Move model_name to wrapper constructor, remove from method params - Update send_gemini_file_batches to use wrapper-level validation - Add tests for model validation logic * feat(ai): add deduplication to send_gemini_file_batches - Skip files already processed with same model + prompts combo - Add --force flag to bypass dedup and reprocess all files - Query LLMTask for SUCCEEDED/FINISHED status keyed on S3 key, model, and both prompts
…elawproject-dependencies Update freelawproject dependencies
Frontend Checks SummaryFound 137 error(s), 9 warning(s). Errors
Warnings
Help: Unstyled linksLinks in redesign templates need Tailwind styling classes. Common fixes:
|
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
16 participants
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.
Fixes
This fixes...
Summary
This PR...
Deployment
This PR should:
skip-deploy(skips everything below)skip-web-deployskip-celery-deployskip-cronjob-deployskip-daemon-deployScreenshots
Desktop
Mobile