plan_my_stuff 0.15.0 → 0.17.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8a3c6c6445f04d677b6d6b4f0219f57326346b3f42aee6839ab98a0bbf680e63
-  data.tar.gz: f5f5adb4f7a11e2d35b907ae7ab1e15d9dc47f31313355be7da186bf9c5086d6
+  metadata.gz: fe4f92dc92eca010b6fc7f53f4ce7701c6852420c9a537fdf0902e591c3309ad
+  data.tar.gz: b167fd42a3b5a37fc897bebf8f3696ed2cd863ac7049359445a329d9a7861988
 SHA512:
-  metadata.gz: a31f3331be41dcd8c4a1369f729888e06ec64474a58c0b56d2c7d57d69b647b1f33b7acc0ae4eed26f03491ce5736a3ab60656900fc31e731bbadd2ae68c492c
-  data.tar.gz: 429281e1a5b431ab29fc5c2d91f15176c7556533df7a94d9990dc3e98a198d5b2d603205bd29f5d4efd210c131bc67b7d440de296857f2eeddd5c085fbc799a2
+  metadata.gz: 746332e19a507f81dc0ee902a87664dcb0a55e3170d688d878b14faf508bfb67cc5dd3236a4961c6b15768fd47e7489b6b0cdbe5bdd5d328bcf9836f805d561d
+  data.tar.gz: 26a20b3318c99f34360bba90eee5698242f4043043991bf53d59e55b2507516452a5094c14366b19f6f53a140ff0b8a41f326a7048c13c4077f447eb2a083513

data/CHANGELOG.md

@@ -1,5 +1,32 @@
 # Changelog
 
+## 0.17.0
+
+### Added
+
+- `PlanMyStuff.deprecator` returns a gem-scoped `ActiveSupport::Deprecation` instance.
+  Consuming apps can silence, filter, or re-route PMS deprecation warnings independently
+  of Rails' own deprecator.
+
+### Changed
+
+- Issue, comment, and project-readme metadata is now stored in a visible `<details>` block
+  with a JSON code fence instead of a hidden HTML comment. GitHub renders the block in-page
+  so the metadata is inspectable without opening the body editor (closes #58).
+  `MetadataParser.parse` still accepts the legacy `<!-- pms-metadata:... -->` format and
+  emits a deprecation warning when it sees one; pre-0.17.0 bodies keep parsing and
+  re-serialize to the new format on their next write. Legacy detection will be removed
+  in 1.0.0.
+
+## 0.16.0
+
+### Added
+
+- `PlanMyStuff::Issue.count(repo:, state:, labels:)` returns the total count of matching issues via the
+  GitHub Search API in a single request, without paginating full issue payloads. Mirrors `Issue.list`
+  kwargs (minus `page`/`per_page`); returns an `Integer`. Excludes PRs server-side via the `is:issue`
+  qualifier.
+
 ## 0.15.0
 
 ### Breaking

data/lib/plan_my_stuff/issue.rb

@@ -312,6 +312,37 @@ module PlanMyStuff
         filtered.map { |gi| build(gi, repo: resolved_repo) }
       end
 
+      # Counts GitHub issues matching the given filters without paginating full payloads.
+      #
+      # Uses GitHub's Search API (+search/issues+), which returns +total_count+ in a single
+      # request. The +is:issue+ qualifier excludes PRs server-side.
+      #
+      # Caveats:
+      # - The search index lags writes by up to ~1 minute, so freshly created/closed issues
+      #   may not be reflected immediately.
+      # - The Search API has its own rate limit (30 req/min authenticated) separate from
+      #   the core REST API.
+      #
+      # @param repo [Symbol, String, nil] defaults to config.default_repo
+      # @param state [Symbol] :open, :closed, or :all
+      # @param labels [Array<String>]
+      #
+      # @return [Integer]
+      #
+      def count(repo: nil, state: :open, labels: [])
+        client = PlanMyStuff.client
+        resolved_repo = client.resolve_repo!(repo)
+
+        normalized_state = state.to_s
+        qualifiers = ["repo:#{resolved_repo}", 'is:issue']
+        qualifiers << "is:#{normalized_state}" unless normalized_state == 'all'
+        labels_to_use = Array.wrap(labels).sort
+        qualifiers += labels_to_use.map do |label|
+          "label:\"#{label}\""
+        end
+        client.rest(:search_issues, qualifiers.join(' '), per_page: 1).total_count
+      end
+
       # Submits one or more pre-built payloads to GitHub's "Import Issues" preview endpoint
       # (+POST /repos/:repo/import/issues+). One request per payload: the endpoint only accepts a single
       # +{issue:, comments:}+ payload at a time.

data/lib/plan_my_stuff/metadata_parser.rb

@@ -1,10 +1,26 @@
 # frozen_string_literal: true
 
+require 'active_support/deprecation'
 require 'json'
 
 module PlanMyStuff
   module MetadataParser
-    METADATA_PATTERN = /\A<!-- pms-metadata:(.*?) -->\n*/m
+    LEGACY_DEPRECATION_MESSAGE =
+      'PlanMyStuff: legacy <!-- pms-metadata: ... --> body format detected. It will continue to parse ' \
+        'until 1.0.0, at which point legacy detection will be removed. New writes already use the visible ' \
+        '<details> block format introduced in 0.17.0; existing bodies migrate on their next write.'
+
+    # New format: collapsible <details> block containing a JSON code fence.
+    # Renders visibly on GitHub (issue #58) instead of being hidden in an HTML comment.
+    METADATA_PATTERN = %r{
+      \A<details><summary>pms-metadata</summary>\n\n
+      ```json\n(.*?)\n```\n\n
+      </details>\n*
+    }mx
+
+    # Legacy format kept for parsing only - existing issues serialized before 0.17.0
+    # used a hidden HTML comment. They migrate to the new format on the next write.
+    LEGACY_METADATA_PATTERN = /\A<!-- pms-metadata:(.*?) -->\n*/m
 
     module_function
 
@@ -17,11 +33,14 @@ module PlanMyStuff
     def parse(raw_body)
       return { metadata: {}, body: '' } if raw_body.blank?
 
-      match = raw_body.match(METADATA_PATTERN)
-      return { metadata: {}, body: raw_body } if match.nil?
+      pattern = matching_pattern(raw_body)
+      return { metadata: {}, body: raw_body } if pattern.nil?
 
+      PlanMyStuff.deprecator.warn(LEGACY_DEPRECATION_MESSAGE) if pattern == LEGACY_METADATA_PATTERN
+
+      match = raw_body.match(pattern)
       metadata = JSON.parse(match[1], symbolize_names: true)
-      body = raw_body.sub(METADATA_PATTERN, '')
+      body = raw_body.sub(pattern, '')
 
       { metadata: metadata, body: body }
     rescue JSON::ParserError
@@ -44,12 +63,21 @@ module PlanMyStuff
 
       json =
         if metadata.is_a?(PlanMyStuff::CustomFields)
-          metadata.to_json
+          JSON.pretty_generate(metadata.to_h)
         else
           JSON.pretty_generate(metadata)
         end
 
-      "<!-- pms-metadata:#{json} -->\n\n#{body}"
+      "<details><summary>pms-metadata</summary>\n\n```json\n#{json}\n```\n\n</details>\n\n#{body}"
+    end
+
+    # @param raw_body [String]
+    #
+    # @return [Regexp, nil] the pattern that matches at the start of +raw_body+, or +nil+ when neither matches
+    #
+    def matching_pattern(raw_body)
+      return METADATA_PATTERN if raw_body.match?(METADATA_PATTERN)
+      return LEGACY_METADATA_PATTERN if raw_body.match?(LEGACY_METADATA_PATTERN)
     end
   end
 end

data/lib/plan_my_stuff/version.rb

@@ -3,7 +3,7 @@
 module PlanMyStuff
   module VERSION
     MAJOR = 0
-    MINOR = 15
+    MINOR = 17
     TINY = 0
 
     # Set PRE to nil unless it's a pre-release (beta, rc, etc.)

data/lib/plan_my_stuff.rb

@@ -38,6 +38,11 @@ module PlanMyStuff
       @configuration ||= PlanMyStuff::Configuration.new
     end
 
+    # @return [ActiveSupport::Deprecation]
+    def deprecator
+      @deprecator ||= ActiveSupport::Deprecation.new('1.0', 'PlanMyStuff')
+    end
+
     # @return [PlanMyStuff::Configuration]
     def configure
       yield(configuration)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: plan_my_stuff
 version: !ruby/object:Gem::Version
-  version: 0.15.0
+  version: 0.17.0
 platform: ruby
 authors:
 - Brands Insurance