refactor: update Makefile and improve README, add pinger functionality

[kazeburo]

User description

• Simplified Makefile to remove unnecessary GITCOMMIT variable.
• Enhanced README with installation instructions and usage examples.
• Refactored main.go to improve error handling and streamline the pinging process.
• Added pinger.go for IP address resolution and pinger creation.
• Removed main_test.go and added pinger_test.go for testing the new pinger functionality.
• Updated go.mod and go.sum to include new dependencies.

---

PR Type

Enhancement, Tests, Documentation

---

Description

• Integrates flagrun for CLI execution

• Refactors IP resolution and pinger creation

• Adds IPv4 and IPv6 pinger tests

• Expands installation, configuration, and metrics documentation

---

Diagram Walkthrough

flowchart LR
  CLI["flagrun CLI execution"] --> PING["Resolve host and create pinger"]
  PING --> METRICS["Collect ping results and RTT metrics"]
  DOCS["README usage documentation"] --> CLI
  TESTS["IPv4 and IPv6 tests"] --> PING

Loading

File Walkthrough

	Relevant files
Enhancement	main.go Integrate flagrun into plugin execution                                    main.go Replaces custom argument parsing with flagrun Implements the Run interface and exit statuses Routes runtime errors to standard error Removes embedded IP resolution and version handling +11/-79  pinger.go Add protocol-aware pinger creation                                              pinger.go Adds reusable host resolution and pinger creation Detects IPv4 and IPv6 from resolved addresses Configures protocol-specific local bind addresses +25/-0
Tests	pinger_test.go Test IPv4 and IPv6 pinger setup                                                    pinger_test.go Tests IPv4 address resolution and pinger creation Tests IPv6 address resolution and pinger creation Skips tests when ICMP privileges are unavailable [link]
Configuration changes	Makefile Simplify builds and add linting                                                    Makefile Removes Git commit metadata from build flags Builds all Go source files Adds lint and expands phony targets +7/-5
Documentation	README.md Expand Japanese usage and metrics documentation                    README.md Adds Japanese installation and usage instructions Documents agent configuration and security permissions Describes emitted metrics and preflight behavior Includes IPv4 and IPv6 support details +64/-3
Dependencies	go.mod Update CLI and networking dependencies                                      go.mod Adds the flagrun dependency Updates stats and networking dependencies Marks command-line parsing dependencies as indirect +7/-4

---

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[github-actions]

PR Reviewer Guide 🔍

(Review updated until commit 0d617a3)

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪

🧪 PR contains tests

🔒 No security concerns identified

⚡ Recommended focus areas for review IPv6 Test Failure The test only skips when pinger creation fails due to permission errors. On IPv4-only systems or CI environments with IPv6 disabled, creating the IPv6 pinger can fail with errors such as EAFNOSUPPORT, causing go test ./... to fail even though the IPv4 functionality works. Handle unsupported IPv6 errors as a skip, or avoid requiring IPv6 support in the default test suite. ipv6Addr := "2001:4860:4860::8888" ipv6IPAddr, ipv6Pinger, err := resolveIPAddrAndPinger(ipv6Addr) if err != nil { if os.IsPermission(err) || errors.Is(err, syscall.EPERM) || errors.Is(err, syscall.EACCES) { t.Skipf("skipping: requires privileges to create ICMP sockets: %v", err) } t.Fatalf("failed to resolve IPv6 address: %v", err)

[github-actions]

Persistent review updated to latest commit 7e9d4e8

[github-actions]

Persistent review updated to latest commit d55162f

[github-actions]

Persistent review updated to latest commit 1e51cd1

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

Persistent review updated to latest commit 0d617a3

[github-actions]

PR Code Suggestions ✨

No code suggestions found for the PR.