Skip to content

labeler#45

Merged
JosunLP merged 13 commits intodevelopmentfrom
JosunLP-patch-1
Mar 12, 2026
Merged

labeler#45
JosunLP merged 13 commits intodevelopmentfrom
JosunLP-patch-1

Conversation

@JosunLP
Copy link
Copy Markdown
Collaborator

@JosunLP JosunLP commented Mar 12, 2026

This pull request introduces several improvements to the bQuery.js project, focusing on documentation, workflow automation, and contributor guidance. The most significant changes are the addition of new rule and instruction files for developers and AI agents, expanded module documentation, and the setup of GitHub workflows for code quality and labeling.

Contributor and AI Agent Guidance:

  • Added .clinerules and .cursorrules files to provide concise, themed rules and quick references for developers and AI agents working on bQuery.js. These documents cover project invariants, module imports, testing patterns, and file structure. [1] [2]
  • Expanded .github/copilot-instructions.md with detailed module descriptions, new command references, usage examples for router, store, and view modules, common pitfalls, and AI agent resources. [1] [2] [3] [4]

GitHub Workflow Automation:

  • Added .github/workflows/eslint.yml to automate ESLint scanning for code quality on pushes, pull requests, and a weekly schedule, including SARIF reporting and integration with GitHub security events.
  • Added .github/workflows/label.yml to automatically label pull requests based on modified paths, streamlining triage and review.

Issue and Feature Request Templates:

  • Introduced .github/ISSUE_TEMPLATE/bug_report.md and .github/ISSUE_TEMPLATE/feature_request.md to standardize issue and feature submissions, improving communication and tracking. [1] [2]

Miscellaneous:

  • Updated .github/FUNDING.yml by commenting out unused funding platform fields for clarity.

These changes collectively enhance project maintainability, contributor onboarding, and code quality assurance.

JosunLP and others added 5 commits February 10, 2026 23:54
* Refactor motion module: Split functionality into separate files and enhance type definitions

- Moved transition, flip, spring, stagger, timeline, keyframes, scroll, and reduced motion functionalities into their respective files for better organization and maintainability.
- Introduced new types for better type safety and clarity across the motion module.
- Added keyframe presets for common animations and scroll-triggered animation helpers.
- Implemented a prefers-reduced-motion detection utility.
- Enhanced tests to cover new functionalities and ensure robustness.

* Füge Sicherheitsmodule hinzu: Modularisiere interne Logik, implementiere Trusted Types und verbessere die HTML-Sanitierung

* - **Router**: Internals modularized into focused submodules with no public API changes.

### Fixed (Unreleased)

- **Security**: `security/sanitize` now re-exports `generateNonce()` and `isTrustedTypesSupported()` for legacy deep imports.

* Füge defineComponent-Funktion hinzu und modularisiere Komponentenlogik in Submodule

* Füge linkedSignal-Funktion hinzu: Erstelle schreibbare berechnete Werte, die Getter und Setter verknüpfen

* feat(store): implement devtools integration and refactor store structure

- Add devtools integration for store management with functions to register, unregister, and notify state changes.
- Refactor store module to separate concerns into distinct files: create-store, mapping, persisted, plugins, registry, types, utils, and watch.
- Introduce mapping helpers for state, getters, and actions to simplify component integration.
- Implement persistence functionality for stores using localStorage.
- Add watch functionality to observe state changes with customizable options.
- Enhance tests to cover new features including mapGetters and watchStore.

* feat(view): implement directive handlers for dynamic binding

- Add handleStyle directive for dynamic style binding in style.ts
- Add handleText directive for setting text content in text.ts
- Create evaluate function for expression evaluation in evaluate.ts
- Refactor view module by moving mount and template creation logic to mount.ts
- Introduce processElement and processChildren functions for directive processing in process.ts
- Define types for binding context, mount options, and directive handlers in types.ts
- Update index.ts to export new module structure and types

* feat(core): add attribute helpers and collection DOM manipulation methods

* feat(core): modularize utility functions into focused helper modules

* feat(component): sanitize markup in defineComponent to enhance security

* feat(security): sanitize component render markup for Shadow DOM consistency
fix(docs): correct event section heading in Core API guide for BQueryElement

* feat(release): bump version to 1.3.0 and update changelog

* feat(docs): update module sizes in README and add module overview in getting started guide

* feat(security): enhance mXSS detection in sanitizeHtmlCore by verifying stability through double-parsing

* Potential fix for code scanning alert no. 9: DOM text reinterpreted as HTML

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>

* Fix PR review issues: duplicate keys, cleanup leaks, export conflicts, and SSR safety (#17)

* Initial plan

* fix: address PR review comments - duplicate keys, cleanup, exports, unwrap, caching, routing, SSR, directives, and test

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* fix: improve duplicate key warning message to be more accurate

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* test: add coverage for PR fixes - unwrap siblings, defineStore caching, ref cleanup, duplicate keys

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* test: improve duplicate key test with try-finally and reduce duplication

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* docs: clarify unwrap() behavior for all children vs collection-only elements

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Update src/router/router.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update src/core/index.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: Jonas Pfalzgraf <info@josunlp.de>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* feat(security): implement safe HTML parsing with DOMParser and enhance mXSS detection

* Potential fix for code scanning alert no. 12: DOM text reinterpreted as HTML

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>

* Fix security, type safety, and API surface issues from PR #16 review (#18)

* Initial plan

* fix: address PR review comments - router guards, motion kebab-case, component styles, utils exports, docs

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* feat(security): enhance HTML parsing with input normalization and early return for plain text

* feat(security): implement dedicated HTML parsing helper to enhance safety and clarity

* Fix router cancellation, component example, and ref directive expression handling (#19)

* Initial plan

* fix(router): restore full URL with query/hash and use replaceState on cancelled navigation

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* docs(component): update example to use addEventListener instead of inline onclick

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* fix(view): use rawValue for object refs in bq-ref directive to support expressions

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* test: add coverage for nested bq-ref expressions and router query/hash restoration on cancelled navigation

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* fix(component): use event delegation in example to survive re-renders; test: add search/hash assertions

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* docs(component): add disconnected hook to example for proper event listener cleanup

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Fix router compatibility, isDev default, style cleanup, and changelog format (#20)

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* Fix hash routing bugs and add comprehensive test coverage (#21)

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* Fix router base path handling, bq-if DOM stability, and CHANGELOG consistency (#22)

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* Add JsDelivr badge to README (#23)

* Fix defineComponent tests, store reactivity leak, view directive signal handling, and hash-routing interception (#24)

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* Fix async handling, type safety, environment guards, and add test coverage from PR #16 review (#25)

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* Fix DOM order reversal, stale dependency tracking, localStorage SafariError, and reactive bq-for updates (#26)

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* Fix view template root validation, store optimizations, and utils API cleanup (#27)

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Jonas Pfalzgraf <info@josunlp.de>

* Fix bq-show display restoration and timeline.seek() time calculation (#28)

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* Fix router base path duplication and add SSR guards for Element checks (#29)

* Initial plan

* Fix router base path duplication and add SSR guards for Element checks

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Fix SSR safety for interceptLinks default parameter and null router handling

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Address PR review comments: add window guard, fix base="/" edge case, clarify error comments, and add test coverage

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Fix window guard short-circuit and remove trailing whitespace

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Fix interceptLinks() browser behavior and createTemplate() validation (#30)

* Initial plan

* Fix interceptLinks() to respect browser behaviors and improve createTemplate() validation

- Add guards for standard browser behaviors in interceptLinks():
  * Only intercept left-clicks (button 0)
  * Respect Ctrl/Cmd/Shift/Alt modifier keys for new tabs
  * Skip already-prevented events
  * Check for MouseEvent type
- Add proper validation in createTemplate() to ensure exactly one root element
- Note: Did not change beforebegin reversal behavior as current implementation is correct

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Simplify createTemplate validation logic

Remove redundant check - since we already verified el exists, we only need to check for multiple roots

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Improve comment clarity in createTemplate validation

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Improve comment accuracy in validation and event handling

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Simplify comment for better clarity

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Add test coverage for browser behavior guards and template validation

- Add tests for interceptLinks() modifier key guards (Ctrl, Cmd, Shift, Alt)
- Add tests for interceptLinks() button checks (middle-click, right-click)
- Add test for interceptLinks() defaultPrevented check
- Add tests for createTemplate() multiple root element validation
- Add test for createTemplate() with whitespace handling
- All 422 tests pass

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Fix bq-class bracket detection, once() failure caching, mount bq-for validation, and component pre-mount renders (#31)

* Initial plan

* Fix bq-class array detection, once() failure handling, mount bq-for guard, and component pre-mount renders

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Add test coverage for once(), bq-class bracket access, mount bq-for validation, and component hasMounted

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Fix trailing whitespace in view tests

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Enhance prop coercion error handling, add Web Animations API check in timeline, and improve watch options with immediate and custom equality support

* Refactor tests to improve error message handling and update bq-class syntax; remove redundant bq-for test case

* Add GitHub Actions workflow for testing with Bun

* Potential fix for code scanning alert no. 16: Workflow does not contain permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>

* Refactor import statements for consistency, enhance untrack functionality to prevent dependency tracking for computed values, and add related tests

* Refactor imports for consistency, enhance Actions type to include getters, and clean up whitespace in various files

* Update warning message in bq-for tests to reflect index-based key fallback

* Implement lazy context proxy for signal unwrapping in evaluate function

* Format YAML and TypeScript files for consistency and readability

* Add early completion for flip animation on zero dimensions or no change

* Optimize evaluation functions by implementing caching for compiled expressions

* Füge Typdefinitionen für die Sicherheitsfunktionen hinzu und optimiere die Typumwandlung in der Prop-Koerzierungsfunktion

* Optimiere den Cache für evaluateRaw-Funktionen und verbessere die Schlüsselgenerierung

* Update README.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Verbessere die Handhabung von Funktionsreferenzen in der bq-on:event-Direktive und optimiere die Cache-Logik in defineStore

* Verbessere die Typisierung und Validierung in der generateNonce-Funktion, füge Chunking für die String-Erstellung hinzu; aktualisiere die truncate-Dokumentation und verbessere die Proxy-Implementierung in createLazyContext

* Füge eine LRU-Cache-Implementierung für die Auswertung von Ausdrücken hinzu und verbessere die Cache-Verwaltung; implementiere Tests für die Cache-Leerung

* Verbessere die Handhabung von Leerzeichen in der class-Direktive und aktualisiere die Bun-Version in der CI-Konfiguration

* Aktualisiere die Bun-Version auf 1.3.7 in den CI-Konfigurationen und verbessere die Fehlerbehandlung beim Erstellen der Trusted Types-Policy.

* Verbessere die Handhabung der erforderlichen Eigenschaften im defineComponent und führe eine verzögerte Initialisierung des Mounts durch; aktualisiere die Typdefinitionen für die merge-Funktion zur Unterstützung von bis zu 5 Quellen.

* Füge Fallback für SSR/non-DOM-Umgebungen in der transition-Funktion hinzu und verbessere die Handhabung von localStorage in den Tests.

* Fix bq-style whitespace handling and spring variable frame rate (#32)

* Initial plan

* Fix bq-style whitespace handling and spring variable frame rate

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Improve documentation for spring variable frame rate timing

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Add test for bq-style directive with leading whitespace

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Fix bq-on method reference context loss and remove style attribute from security defaults (#33)

* Initial plan

* Fix bq-on method reference docs and remove style from default allowed attributes

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Fix persistedSignal localStorage test key collision (#34)

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* Fix bq-class directive stale class cleanup for object syntax (#35)

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Jonas Pfalzgraf <info@josunlp.de>

* Fix timeline() to account for animation iterations in duration calculations (#36)

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* Fix createTemplate to reject templates with bq-if directive on the root element

* Fix test job permissions in GitHub Actions workflow

* Fix timeline to conditionally apply final keyframe styles based on commitStyles flag

* Fix createRouter to correctly handle base path matching for navigation

* 🔀 Update CHANGELOG.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Fix handleShow to ensure cross-document compatibility for computed display values

* Enhance prop coercion by adding explicit control for constructor invocation in PropDefinition

* Update devDependencies to latest versions for improved compatibility

* chore: update dependencies in package.json and improve test readability

- Updated eslint-config-prettier from ^9.1.2 to ^10.1.8
- Updated globals from ^15.15.0 to ^17.2.0
- Updated rimraf from ^5.0.10 to ^6.1.2
- Updated typedoc from ^0.25.13 to ^0.28.16
- Updated vite from ^5.4.21 to ^7.3.1
- Cleaned up whitespace in view.test.ts for better readability

* refactor: simplify logic for ordering elements in insertContent function

* refactor: enhance type handling in handleRef for improved clarity and safety

* refactor: clarify logic for reversing element order in insertContent function

* docs: update documentation for new features and improvements across modules

* fix: Reactive robustness, security hardening, storage compat, expression parsing (#37)

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Jonas Pfalzgraf <info@josunlp.de>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* Fix css() getter, add Signal.dispose(), debounce/throttle cancel, collection.find(), element.is() (#39)

* Initial plan

* fix: css() getter, add Signal.dispose(), effect error handling, debounce/throttle cancel, is(), find()

- Fix BQueryElement.css() to return computed style when used as getter
- Fix BQueryCollection.css() to return computed style from first element
- Add error handling in effect() to catch errors without breaking reactive graph
- Add Signal.dispose() method to clear all subscribers
- Add cancel() method to debounce() and throttle() return values
- Add BQueryElement.is() as jQuery-compatible alias for matches()
- Add BQueryCollection.find() for descendant querying
- Add getComputedStyle to test setup for happy-dom
- Add comprehensive tests for all changes

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* perf: use Set for deduplication in BQueryCollection.find()

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* refactor: address review feedback - delegate is() to matches(), use ownerDocument.defaultView for getComputedStyle, update css() JSDoc

- is() now delegates to this.matches() instead of duplicating the implementation
- css() getter uses ownerDocument.defaultView?.getComputedStyle() for cross-document/SSR safety
- Updated css() JSDoc @returns to describe getter/setter return types
- Same fix applied to BQueryCollection.css() getter

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* fix: properly type debounced/throttled via Object.assign, add getComputedStyle guard in collection.css()

- Use Object.assign to create DebouncedFn/ThrottledFn with proper typing
- Add typeof getComputedStyle check in BQueryCollection.css() getter for SSR safety

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* docs: add css() getter example to JSDoc, strengthen effect error test

- Add getter usage example to BQueryElement.css() JSDoc
- Verify effect continues working after catching an error

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* fix: use strict undefined check for debounce timeoutId instead of truthiness

Handles edge case where setTimeout could return 0 in browser environments.

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* fix: implement removeDependency to prevent stale references on signal disposal

* fix: update css() method signatures for better type safety

* fix: update css() method signatures for improved type handling

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* chore: update dependencies and version, improve code readability in evaluate.ts and security tests

* fix: update funding model comments for clarity and consistency

* feat: add Cline and Cursor rules, enhance Copilot instructions, and update AGENT.md for AI agent support
feat: implement new features in core API including css() getter, is() method, find() method, and debounce/throttle enhancements
fix: improve reactive system error handling and add Signal.dispose() method
fix: enhance security validations for srcset and form action attributes
docs: update README.md and API documentation with new features and usage examples
chore: add llms.txt for LLM optimization and project summary

* fix: update security rule to reference correct sanitize module path

* fix: add optional chaining to prevent errors when accessing defaultView
fix: reset timeoutId in debounce function to avoid unintended behavior

* fix: clarify srcset validation behavior in documentation and examples

* Update tests/view.test.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Add ESLint workflow for code analysis

This workflow runs ESLint to scan JavaScript and TypeScript code on push and pull request events, and uploads the results in SARIF format.

* Update .github/workflows/eslint.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Change npm to bun for dependency installation

* Replace npx with bunx for ESLint execution

* Add bun installation to ESLint workflow

* Install bun globally in ESLint workflow

Changed npm install command to install bun globally.

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This workflow triages pull requests and applies labels based on modified paths. Configuration is required in .github/labeler.yml.

Signed-off-by: Jonas Pfalzgraf <info@josunlp.de>
@JosunLP JosunLP self-assigned this Mar 12, 2026
Copilot AI review requested due to automatic review settings March 12, 2026 14:32
@JosunLP
Copy link
Copy Markdown
Collaborator Author

JosunLP commented Mar 12, 2026

@copilot since we introduce a label action, generate fitting label definitions for the action to use for the project

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 12, 2026

@JosunLP I've opened a new pull request, #46, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates bQuery.js to v1.4.0 with new core/reactive features and fixes (CSS getter semantics, cancellable debounce/throttle, safer reactive error handling, improved sanitization), plus expanded documentation and new GitHub automation for linting and PR labeling.

Changes:

  • Core API enhancements: css() getter overloads, BQueryElement.is(), BQueryCollection.find(), and cancellable debounce()/throttle() (with new exported types).
  • Reactive + security + platform fixes: safer batching/effect error handling, Signal.dispose(), per-URL srcset sanitization, form[action] URL validation, and spec-compliant web storage keys().
  • Project automation/docs: new/updated docs + agent guidance files, plus new ESLint + labeler workflows and GitHub issue templates.

Reviewed changes

Copilot reviewed 33 out of 36 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
tests/view.test.ts Adds tests for parseObjectExpression edge cases.
tests/utils.test.ts Adds tests for debounce.cancel() and throttle.cancel().
tests/signal.test.ts Adds tests for reactive error handling, batch robustness, and Signal.dispose().
tests/setup.ts Exposes getComputedStyle globally for new CSS getter tests.
tests/security.test.ts Adds tests for srcset URL validation and form[action] URL validation.
tests/platform.test.ts Adds coverage for storage.local().keys() behavior.
tests/core.test.ts Adds tests for css() getter/setter behavior, is(), and collection.find().
src/view/evaluate.ts Fixes quote-escape parsing in parseObjectExpression by counting backslashes.
src/security/sanitize-core.ts Validates srcset per-URL and adds URL validation for action.
src/reactive/internals.ts Makes batch flushing resilient to observer errors; guards endBatch() underflow; adds removeDependency().
src/reactive/effect.ts Catches/logs effect + cleanup errors to keep the reactive system running.
src/reactive/core.ts Adds Signal.dispose() and dependency cleanup via removeDependency().
src/platform/storage.ts Implements spec-compliant Storage.key(i) iteration for keys().
src/core/utils/function.ts Adds cancellable debounce/throttle return types and implementations.
src/core/index.ts Exports DebouncedFn / ThrottledFn types from the core entry point.
src/core/element.ts Introduces css() getter overload and adds is() alias for matches().
src/core/collection.ts Introduces css() getter overload and adds find() across collection elements with dedupe.
package.json Bumps version to 1.4.0 and updates devDependency versions.
llms.txt Adds LLM-oriented project summary for agents/tools.
docs/guide/security.md Documents srcset and form[action] sanitization behavior.
docs/guide/reactive.md Documents Signal.dispose() and effect error handling behavior.
docs/guide/api-core.md Updates core API docs for css() getter, is(), find(), and cancellable debounce/throttle.
bun.lock Updates lockfile for dependency bumps.
README.md Updates examples and highlights new APIs + AI agent support files.
CHANGELOG.md Adds 1.4.0 release notes covering the new APIs and fixes.
AGENT.md Adds comprehensive AI coding agent guidance and repo conventions.
.github/workflows/label.yml Adds PR labeler workflow.
.github/workflows/eslint.yml Adds ESLint SARIF workflow for GitHub code scanning.
.github/copilot-instructions.md Expands Copilot context with module docs, commands, and pitfalls.
.github/ISSUE_TEMPLATE/feature_request.md Adds feature request issue template.
.github/ISSUE_TEMPLATE/bug_report.md Adds bug report issue template.
.github/FUNDING.yml Comments out unused funding fields for clarity.
.cursorrules Adds Cursor-specific repo rules/quick references.
.clinerules Adds Cline-specific repo rules/quick references.

JosunLP and others added 3 commits March 12, 2026 15:40
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Jonas Pfalzgraf <jolun.swtor@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Jonas Pfalzgraf <jolun.swtor@gmail.com>
* Initial plan

* Add PR label definitions

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>

* Update labeler.yml

Signed-off-by: Jonas Pfalzgraf <info@josunlp.de>

---------

Signed-off-by: Jonas Pfalzgraf <info@josunlp.de>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: Jonas Pfalzgraf <info@josunlp.de>
@JosunLP
Copy link
Copy Markdown
Collaborator Author

JosunLP commented Mar 12, 2026

@copilot open a new pull request to apply changes based on the comments in this thread

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 12, 2026

@JosunLP I've opened a new pull request, #47, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 2 commits March 12, 2026 16:08
Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Signed-off-by: Jonas Pfalzgraf <info@josunlp.de>
@JosunLP
Copy link
Copy Markdown
Collaborator Author

JosunLP commented Mar 12, 2026

@copilot Actions are failing; fix it here

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 12, 2026

@JosunLP I've opened a new pull request, #48, to work on those changes. Once the pull request is ready, I'll request review from you.

)

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Jonas Pfalzgraf <info@josunlp.de>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 7 changed files in this pull request and generated 5 comments.

@JosunLP
Copy link
Copy Markdown
Collaborator Author

JosunLP commented Mar 12, 2026

@copilot open a new pull request to apply changes based on the comments in this thread

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 12, 2026

@JosunLP I've opened a new pull request, #49, to work on those changes. Once the pull request is ready, I'll request review from you.

…mples (#49)

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
@JosunLP
Copy link
Copy Markdown
Collaborator Author

JosunLP commented Mar 12, 2026

@copilot eslint action is failing

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 12, 2026

@JosunLP I've opened a new pull request, #50, to work on those changes. Once the pull request is ready, I'll request review from you.

#50)

Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

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

@JosunLP JosunLP merged commit 55b23e0 into development Mar 12, 2026
13 checks passed
@JosunLP JosunLP deleted the JosunLP-patch-1 branch March 12, 2026 16:29
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.

3 participants